Skip to content

Deprecate Forecast Mod (forecastMod.py) - #208

Open
mdeshotel wants to merge 2 commits into
developmentfrom
forecastMod_refactor
Open

Deprecate Forecast Mod (forecastMod.py)#208
mdeshotel wants to merge 2 commits into
developmentfrom
forecastMod_refactor

Conversation

@mdeshotel

@mdeshotel mdeshotel commented Jul 31, 2026

Copy link
Copy Markdown

Refactor Forecast Mod.

EDIT by @mxkpp: due to latest priorities, this will not receive a significant refactor, but the initial improvements here are worth merging.

EDIT by @mxkpp: this file forecastMod.py appears to be unused by the current ngen stack. This file will be deprecated by this PR and it will raise a NotImplementedError if another codebase attempts to import it or run it.

Additions

Removals

Changes

Testing

Screenshots

Notes

Todos

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows project standards (link if applicable)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

  • Linux

@mdeshotel
mdeshotel marked this pull request as draft July 31, 2026 14:46
@mxkpp
mxkpp changed the base branch from master to development July 31, 2026 15:56
@mxkpp
mxkpp force-pushed the forecastMod_refactor branch from c6a3973 to 92367af Compare August 11, 2026 18:27
@mxkpp

mxkpp commented Aug 11, 2026

Copy link
Copy Markdown

Rebased

@mxkpp mxkpp changed the title Forecast mod refactor Refactor of Forecast Mod (forecastMod.py) Aug 11, 2026
@mxkpp mxkpp changed the title Refactor of Forecast Mod (forecastMod.py) Reformat of Forecast Mod (forecastMod.py) Aug 11, 2026
@mxkpp mxkpp changed the title Reformat of Forecast Mod (forecastMod.py) Format, Style, Type Hints of Forecast Mod (forecastMod.py) Aug 11, 2026
@mxkpp
mxkpp self-requested a review August 11, 2026 19:11

@mxkpp mxkpp left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look fine, they don't appear to affect any business logic, and the tests run fine. Approving after making 2 minor adjustments.

@mxkpp
mxkpp marked this pull request as ready for review August 11, 2026 19:17
@mxkpp mxkpp changed the title Format, Style, Type Hints of Forecast Mod (forecastMod.py) Deprecate Forecast Mod (forecastMod.py) Aug 11, 2026
@mxkpp
mxkpp force-pushed the forecastMod_refactor branch from ec9449d to c61e12d Compare August 11, 2026 19:35
@mxkpp

mxkpp commented Aug 11, 2026

Copy link
Copy Markdown

I don't see references to the file forecastMod.py or its function process_forecasts anywhere in the current ngen stack. I think it is actually legacy code that is no longer needed. I have updated this PR to deprecate the entire file by raising a NotImplementedError if someone attempts to use it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants