-
Notifications
You must be signed in to change notification settings - Fork 143
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
Add support for particleset.fieldset as a uxarray object #1883
Comments
I've started really re-working this in the https://github.com/OceanParcels/Parcels/tree/feature/uxarray_xarray_fields branch. At the moment, most of the work is being done in the xfield.py module. This module defines the Additionally, I'm working on preserving some of the functionality from the original After reviewing xarray and uxarray documentation and playing around with creating sample xarray and uxarray datasets for the stommel gyre configuration, it became clear that leveraging field and fieldset as thin wrappers around the |
@erikvansebille and @VeckoTheGecko - I've started summarizing some of the changes I've made (and am planning) in this hackmd document Any feedback is welcome. |
Overall, I think that looks good (namely the attributes available, and considerations on units and time extrapolation - didn't look into the unstructured naming conventions) . I can imagine this changing somewhat as development progresses |
I agree. This has been slowly evolving as I'm working on the implementation and the stommel examples. |
I agree that it looks good! I also added a comment about the |
As it's more pythonic to attach UnitConverters directly to a field? See also #1883 (comment)
Make changes to the particleset to handle the scenario when the
fieldset
is auxarray
object.The text was updated successfully, but these errors were encountered: