-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Publish decadal fixes to new c3s indices #92
Comments
Hi @ellesmith88, I tested the fixes in daops and noted that the |
@agstephens I agree with you and this should be straightforward to implement now that we are deriving values in the fixes. However I will have to regenerate all the fixes (which should hopefully be a lot quicker this time!). I think it must be leftover from when I was using the example file to create the fix template. If I am regenerating the fixes, should I also remove the fill value fix as I mentioned in roocs/daops#88? As it is all done in clisops now. |
@ellesmith88 Thanks, I just responded to roocs/daops#88 so you should be ready to proceed with this. |
Awaiting new version after early 2022 fixes are updated and published. |
Updated fixes are currently being indexed 😄 |
Fixes have been updated in |
@ellesmith88 are the fixes all in place now? Thanks |
@agstephens Yes all fixes are in the index and |
Hi @ellesmith88, the fixes are looking good. Great stuff. |
Tested with local script:
Script is:
|
Publish decadal fixes to c3s-roocs-fix-proposal and c3s-roocs-fix indices.
This involves understanding what has changed with date/text field for value.
Also need to add the fix for removing the coordinate attribute (in dachar and daops) - now that the xarray fix has been released in v0.19
The text was updated successfully, but these errors were encountered: