xMIP
xMIP copied to clipboard
Add `longitude_bnds` and `latitude_bnds` to `cmip_renaming_dict`
Add longitude_bnds
and latitude_bnds
to cmip_renaming_dict
, this allows renaming for example renaming bounds for 'ScenarioMIP.UA.MCM-UA-1-0.ssp585.r1i1p1f2.Amon.gn.none.tas'
.
- [x] Tests added
- [x] Passes
pre-commit run --all-files
- [x] Passes
pytest
- [x] User visible changes (including notable bug fixes) are documented in
whats-new.rst
Thank you very much for adding these @JoranAngevaare. I think the testing failure is unrelated (looks like #301). Let me take a crack at that and see if that resolves the CI failures here. Sorry for the delay.
Ok I am fairly confident that the above issue is fixed by https://github.com/jbusecke/xarrayutils/pull/162, since the upstream-dev
tests are now passing. Once v2.0.0
is published on conda-forge I expect the other tests to pass.
Ok I got it all sorted. Thanks for your patience @JoranAngevaare.
Would you mind adding an entry to the docs/whats-new.rst
? Once that is done, ill merge this right away!
@jbusecke thanks a lot for all the fixes! I see that numpy 1.24.0 drop of deprecated np.float
caused some issues, the np.zeros(.., float)
will be interpreted as np.zeros(.., np.float64)
's which would also have solved that particular problem.
I added a trivial test eff95dc, however, that lead me to notice that bnds
is renamed to bnds
, I'm not entirely sure if that is a desirable feature. If so, I'll very quickly revert a6a5835.
that lead me to notice that
bnds
is renamed tobnds
I am not sure I follow this. I do intent to rename the bounds
dimension similarly to others, but I might be missing something in your comment.
Hi @JoranAngevaare again sorry for the delay (the amount of times I have typed this today 😆). I am not quite following myself from the past TBH. I now completely agree that renaming "bnds" to "bnds" is nonsensical. Sorry for the back and forth. Could you restore that change? Ill get this moving right after the tests pass.
I would also like to add you to the CITATION.cff for the contributions you made here.
@JoranAngevaare finally getting around to merging this. Again sorry for the delay. I have opened #358 to track the addition to the CITATION file.
hi @jbusecke sorry for the delay on my side - I missed these previous comments.