axiom-data-science / extract_model

Facilitates read-in and extraction for ocean model output.
https://extract-model.readthedocs.io/
MIT License
5 stars 4 forks source link

Added xoak functionality #56

Closed kthyng closed 2 years ago

kthyng commented 2 years ago

sel2d functionality is now done by xoak which is faster and better. argsel2d is no longer needed. Other changes are propagation of these changes or tests.

codecov[bot] commented 2 years ago

Codecov Report

Merging #56 (5f4f939) into main (8239d4f) will increase coverage by 10.50%. The diff coverage is 87.36%.

@@             Coverage Diff             @@
##             main      #56       +/-   ##
===========================================
+ Coverage   72.06%   82.56%   +10.50%     
===========================================
  Files           8       10        +2     
  Lines         562      912      +350     
===========================================
+ Hits          405      753      +348     
- Misses        157      159        +2     
Flag Coverage Δ
unittests 82.56% <87.36%> (+10.50%) :arrow_up:

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

Impacted Files Coverage Δ
extract_model/accessor.py 90.74% <61.53%> (-2.71%) :arrow_down:
extract_model/extract_model.py 75.51% <73.33%> (+3.54%) :arrow_up:
tests/test_accessor.py 91.66% <91.66%> (ø)
extract_model/__init__.py 81.81% <100.00%> (ø)
tests/test_em.py 98.75% <100.00%> (+0.18%) :arrow_up:
tests/test_sel2d.py 100.00% <100.00%> (ø)
tests/test_utils.py 100.00% <0.00%> (ø)
extract_model/utils.py 62.89% <0.00%> (+29.56%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8239d4f...5f4f939. Read the comment docs.

kthyng commented 2 years ago

Can't figure out what is wrong with docs. Linting has to be fixed in the main branch.