hainegroup / oceanspy

A Python package to facilitate ocean model data analysis and visualization.
https://oceanspy.readthedocs.io
MIT License
98 stars 32 forks source link

Iss324 #328

Closed Mikejmnez closed 1 year ago

Mikejmnez commented 1 year ago

closes #324 and #327

Data (field variables) of the LLC4320 have been copied into a single zarr store that is distributed across filedbs. Now,

path = '/home/idies/workspace/poseidon/data*/llc4320_tests/10dayhourly/velocities'
ds = xr.open_mfdataset(path, parallel=True, engine='zarr')

reads all data variables. The grid remains separate to avoid copies within each store.

There should be an improvement on how fast the od associated with LLC4320 gets created...

codecov[bot] commented 1 year ago

Codecov Report

Merging #328 (e72a7bf) into main (0bef97d) will increase coverage by 0.00%. The diff coverage is 80.00%.

@@           Coverage Diff           @@
##             main     #328   +/-   ##
=======================================
  Coverage   94.62%   94.62%           
=======================================
  Files           9        9           
  Lines        3942     3944    +2     
  Branches      857      859    +2     
=======================================
+ Hits         3730     3732    +2     
  Misses        135      135           
  Partials       77       77           
Flag Coverage Δ
unittests 94.62% <80.00%> (+<0.01%) :arrow_up:

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

Impacted Files Coverage Δ
oceanspy/llc_rearrange.py 82.67% <ø> (ø)
oceanspy/subsample.py 97.71% <80.00%> (+<0.01%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Mikejmnez commented 1 year ago

Actually, I am making a small change in the next 1/2 hr. Hold on before merging :)