Skip to content

Config files in output file attributes - #567

Merged
ppinchuk merged 29 commits into
mainfrom
pp/config_in_meta
Aug 26, 2025
Merged

Config files in output file attributes#567
ppinchuk merged 29 commits into
mainfrom
pp/config_in_meta

Conversation

@ppinchuk

Copy link
Copy Markdown
Collaborator

reV now stores the config file as a file attribute if/when reV is run from the command line. This should help with traceability and reproducibility of results.

@ppinchuk ppinchuk added this to the FY25 SS milestone Aug 13, 2025
@ppinchuk ppinchuk self-assigned this Aug 13, 2025
@ppinchuk ppinchuk added enhancement Update to logic or general code improvements p-high Priority: high labels Aug 13, 2025
@codecov-commenter

codecov-commenter commented Aug 13, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.94595% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.89%. Comparing base (1f92f0e) to head (5d68619).

Files with missing lines Patch % Lines
reV/supply_curve/aggregation.py 40.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #567      +/-   ##
==========================================
+ Coverage   79.66%   79.89%   +0.22%     
==========================================
  Files          63       63              
  Lines       11254    11294      +40     
  Branches     1617     1621       +4     
==========================================
+ Hits         8966     9023      +57     
+ Misses       1793     1773      -20     
- Partials      495      498       +3     
Flag Coverage Δ
unittests 79.89% <95.94%> (+0.22%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@WilliamsTravis

Copy link
Copy Markdown
Collaborator

Excellent! I've wanted this for a while.

@ppinchuk
ppinchuk merged commit 77c6b06 into main Aug 26, 2025
11 checks passed
@ppinchuk
ppinchuk deleted the pp/config_in_meta branch August 26, 2025 20:15
github-actions Bot pushed a commit that referenced this pull request Aug 26, 2025
Config files in output file attributes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Update to logic or general code improvements p-high Priority: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rep Profiles crashes if cf_dset not given Better attrs tracking Track configs in H5 files

3 participants