-
Notifications
You must be signed in to change notification settings - Fork 34
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
Improve performance of GeoStack and GeoSeries by sharing and caching dims #64
Comments
This could also involve inheriting from However - it's a special case where |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When lazily loading
GeoSeries
the dims of the child files are being loaded/calculated for every file. This should be done once for the first file and copied for the rest. The assumption of the series is that all dims are the same.This may break when it comes to attached metadata being different - that may need some subtlety to handle correctly.
GeoStack
is also not sharing dims between layers in all cases. This is complicated by the fact that not all layers have the same dimensions - but they could choose from a list of shared dimensions, as is done internally in netcdf files.The text was updated successfully, but these errors were encountered: