Repository navigation
test: isolate EP output directories for parallel runs - #1728
Merged
Merged
Conversation
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
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.
EP tests sharing the same output root could delete another worker's figures. Give each graphical test a tmp_path output root, restore config afterward, and remove two decorators that forced a shared temp directory. Model identifiers are unchanged.
Closes #1726.
Validation
Full library suite: 3913 passed, 1 skipped, 1 xfailed. Graphical suite: 329 passed serially and 329 passed with four xdist workers; unchanged main reproduced 13 failures / 316 passed under xdist. Python 3.12.14. Author diff review passed; production code and output identifier generation untouched.
No public API changes. Workspace smoke not applicable to these test-only changes; no science or production path changed. Required Python 3.12/3.13 CI remains pending at PR creation.
Heart and authorization
User authorized "Find 4 grouped tasks and do them", then "Yes I authorise" after the RED gate was disclosed. Development PR only; no merge or release. Fresh published vitals: organism RED, PyAutoFit scope GREEN; local readiness STALE (fresh checkout has no release evidence). This does not repair or clear the organism's RED.
Disclosed RED reasons