Skip to content

Conversation

@amoodie
Copy link
Member

@amoodie amoodie commented Oct 20, 2025

The behavior of the legacy NetCDF output format option, and the new file output format is expected to change in version 2.2.0.

With v2.2.0 the default output file comply with the sandsuet data specification, and the legacy_output=True` option will output the current configuration. The core data will not change with v2.2, but the names and attributes of components of the data output is expected change.

I don't expect dropping the current legacy option to affect any users. Current users will be able to maintain the current file format with legacy_netcdf=True for versions moving forward from v2.2. They will be encouraged to adapt scripts to the new format, but we will maintain backwards compatibility.

See sandsuet data specification for expected file format.

@codecov
Copy link

codecov bot commented Oct 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.10%. Comparing base (8a4ab35) to head (80c0dde).
⚠️ Report is 12 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #293      +/-   ##
===========================================
+ Coverage    79.00%   79.10%   +0.10%     
===========================================
  Files           12       12              
  Lines         2643     2637       -6     
===========================================
- Hits          2088     2086       -2     
+ Misses         555      551       -4     

☔ 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.

@amoodie amoodie merged commit 8e3f548 into DeltaRCM:develop Oct 25, 2025
15 of 16 checks passed
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