Skip to content

Add version tracking to input_output.json (closes #245)#269

Open
ygefen wants to merge 2 commits intomainfrom
feat/version-tracking-experiments
Open

Add version tracking to input_output.json (closes #245)#269
ygefen wants to merge 2 commits intomainfrom
feat/version-tracking-experiments

Conversation

@ygefen
Copy link
Copy Markdown
Contributor

@ygefen ygefen commented Apr 14, 2026

Adds align_system/utils/version.py with get_version(), which uses setuptools-scm to produce an exact git-describe string at runtime (e.g. 0.5.10.dev3+g1234567), with a fallback to importlib.metadata for packaged/containerized environments.

The input_output.json format changes from a bare list to:
{"version": "...", "results": [...]}

Integration test expected files are updated to match the new structure, and the "version" line is added to LOG_EXEMPTIONS in run_integration_test.py so the ever-changing version string is skipped during diff comparisons.

Adds align_system/utils/version.py with get_version(), which uses
setuptools-scm to produce an exact git-describe string at runtime
(e.g. 0.5.10.dev3+g1234567), with a fallback to importlib.metadata
for packaged/containerized environments.

The input_output.json format changes from a bare list to:
  {"version": "...", "results": [...]}

Integration test expected files are updated to match the new structure,
and the "version" line is added to LOG_EXEMPTIONS in run_integration_test.py
so the ever-changing version string is skipped during diff comparisons.
@ygefen ygefen requested a review from dmjoy April 14, 2026 02:06
@ygefen ygefen self-assigned this Apr 14, 2026
…ut.json

Moves the version string out of input_output.json (which would have
changed its schema from a list to an object) into a separate meta.json
file written once at the start of each run. This keeps input_output.json
format stable for downstream consumers while still recording the exact
git-describe version alongside every experiment output.

Reverts the expected integration test files and LOG_EXEMPTIONS back to
their original state accordingly.
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.

1 participant