Closed kmuehlbauer closed 4 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 89.91%. Comparing base (
236bafd
) to head (cb3ab83
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@kmuehlbauer let me know when you need a review here!
Thanks @mgrover1, it's still in the works. Combining this with gamic.py. I'll let you know, when this is ready for review.
Sounds good :) exciting stuff!
@mgrover1 This is ready for review. Essentially this is just a heavy refactor of the odim/gamic backend code with added tests.
Test coverage increased by almost 1%. When this is merged I'll try to cover the last missing lines (at least for odim/gamic).
@mgrover1 Pushed some more code which fit's in here. I'll leave this now as is for your review.
Thanks @mgrover1, let's get this refactor in. The next changes won't be that voluminous.
Restructure and add tests for
_OdimH5NetCDFMetadata
/_GamicH5NetCDFMetadata
. Refactor out_H5NetCDFMetadata
as parent class.history.md