chore: remove 2 dead env_vars overrides copied in from autofit_workspace#26
Merged
Conversation
Both of this repo's env_vars.yaml override patterns matched zero files.
`plot/emcee_plotter` and `plot/zeus_plotter` were copied in from
autofit_workspace/config/build/env_vars.yaml lines 26/28, where the identical
patterns still live and still match. Those scripts exist only as
autofit_workspace/scripts/plot/*.py and have never existed in HowToFit, which
contains only chapter_*/tutorial_*.py.
`overrides: []` is written explicitly rather than left as a bare key, because
env_config.py reads `.get("overrides", [])` and an empty key loads as None,
which would raise on iteration.
Behaviour-neutral: for all 21 scripts here the resolved build_env_for_script
dict is identical before and after, since both removed patterns matched zero
files. This repo has no no_run.yaml entries, so that file is untouched.
Closes part of PyAutoLabs/HowToLens#46
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011ZVaiShnRkv1xbQBegGK1N
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Both of this repo's
env_vars.yamloverride patterns matched zero files.plot/emcee_plotterandplot/zeus_plotterwere copied in fromautofit_workspace/config/build/env_vars.yamllines 26/28, where the identical patterns still live and still match. Those scripts exist only asautofit_workspace/scripts/plot/*.pyand have never existed in HowToFit, which contains onlychapter_*/tutorial_*.py— there is no deletion for them anywhere in this repo's history.This is the same copy-paste mechanism found in HowToLens and HowToGalaxy, making the finding unanimous across all three HowTo repos. Part of a 5-repo census (PyAutoLabs/HowToLens#46) that found 50 of 122
no_run.yamlentries (41%) dead across the organism, plus 11 deadenv_varspatterns.This repo's
no_run.yamlhas no entries at all and is untouched.overrides: []is written explicitly rather than left as a bare key, becauseenv_config.py:73reads.get("overrides", [])and an empty key loads asNone, which would raise on iteration.Scripts Changed
None — this PR is config-only.
config/build/env_vars.yaml— 2 override patterns →overrides: [](plot/emcee_plotter,plot/zeus_plotter)Test Plan
build_env_for_scriptdict is byte-identical before vs after, because both removed patterns matched zero files.Generated by the PyAutoLabs agent workflow.