You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Core already runs the compiled Energyplan worker in beta, but each installation still starts the retired Python service and the updater refuses Core updates without it. This removes the Python package, container and release workflow, runtime settings, Python shadow, and separate optimizer update controls. Core updates carry Energyplan; Core DP remains the validated fallback and comparison shadow.
Older planner.engine: python settings migrate to Energyplan. New Compose installations start no Python service. An explicit updater -retire-python command checks that Energyplan is healthy, starts a helper from the exact installed updater image with a writable project mount, saves the old Compose files, removes the retired service and its FTW socket wiring, and keeps custom services, image pins and persistent data. Older updaters can install this pair before the old container is retired.
Validation: make verify passed, including the bundled worker integration, and all 541 web tests passed. Browser checks covered the engine choices, removal of the optimizer row and Python controls, and the single Core update channel. Migration tests cover installations without Python, preservation of custom overrides, and Core/updater rollback. The new retirement tests cover both Compose environment formats, volume formats, idempotence, preservation of the data mount, writable helper mounts, and orphan removal within the existing project.
This retires the Python code touched by #1036 and the Python portions of #963 and #971. Their separate Go planning and validation work remains outside this change.
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Left a non-blocking comment; not approved. Cursor Bugbot was present but finished as skipped, so its review is not a usable approval signal. No reviewers were assigned because the matching CODEOWNERS path is the PR author.
Sent by Cursor Approval Agent: Pull Request Router and Approver
The reason will be displayed to describe this comment to others. Learn more.
Stale comment
Left a non-blocking comment; not approved. Cursor Bugbot was present but finished as skipped, so its review is not a usable approval signal. No reviewers were assigned because the matching CODEOWNERS path is the PR author.
Sent by Cursor Approval Agent: Pull Request Router and Approver
The reason will be displayed to describe this comment to others. Learn more.
Left a non-blocking comment; not approved. Cursor Bugbot was present but finished as skipped, so its review is not a usable approval signal. No reviewers were assigned because the matching CODEOWNERS path is the PR author.
Sent by Cursor Approval Agent: Pull Request Router and Approver
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
Core already runs the compiled Energyplan worker in beta, but each installation still starts the retired Python service and the updater refuses Core updates without it. This removes the Python package, container and release workflow, runtime settings, Python shadow, and separate optimizer update controls. Core updates carry Energyplan; Core DP remains the validated fallback and comparison shadow.
Older
planner.engine: pythonsettings migrate to Energyplan. New Compose installations start no Python service. An explicit updater-retire-pythoncommand checks that Energyplan is healthy, starts a helper from the exact installed updater image with a writable project mount, saves the old Compose files, removes the retired service and its FTW socket wiring, and keeps custom services, image pins and persistent data. Older updaters can install this pair before the old container is retired.Validation:
make verifypassed, including the bundled worker integration, and all 541 web tests passed. Browser checks covered the engine choices, removal of the optimizer row and Python controls, and the single Core update channel. Migration tests cover installations without Python, preservation of custom overrides, and Core/updater rollback. The new retirement tests cover both Compose environment formats, volume formats, idempotence, preservation of the data mount, writable helper mounts, and orphan removal within the existing project.This retires the Python code touched by #1036 and the Python portions of #963 and #971. Their separate Go planning and validation work remains outside this change.