-
Notifications
You must be signed in to change notification settings - Fork 42
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
Move glacier-specific tools to a separate package/module #636
Comments
In the meantime, the |
@rhugonnet , thanks for the interesting ideas. Allowing for the fact that I'm an xDEM 'bystander', my initial remarks are as follows:
Thanks also for the heads-up on uncertainty propagation software - will look into this. |
Thanks @atedstone! I was just chatting with @trchudley about this again this morning.
Good point! That's indeed the delicate decision 😅. My thoughts are that, as long as the tools don't require new dependencies (geospatial or numerical), they would all fit very well in the same package, as being both thematically and methodologically very close (cryo + reduction/derivative of pre-processed spatiotemporal data). I also think this would work well because code bases to derive this type of estimates are fairly small, whether it is for instance mass balance (from elevation change + glacier outlines), or frontal ablation (from velocity, ice thickness, elevation change + flux gates and glacier outlines). The hard work is usually pre-processing that georeferenced data (which can be done through other specialized geo-package like xDEM). Once it's only estimation from the temporal stacks of data, it's a fairly small amount of methods/routines. In that sense, estimating glacier mass change, frontal ablation or surface albedo in the same package wouldn't shock me. The same way OGGM integrates different modules for different modelling aspects. It could also be the perfect occasion to build open thematic educational resources surrounding the tools (or valorize existing ones, for instance from Bethan Davies)! (a bit like OGGM-Edu) |
Four years later, we've moved forward a lot, and some tools that made sense in the beginning don't fit so well in the package anymore! They should not be discarded, rather moved somewhere that makes more sense, and where we can keep building on top of what we already have 🙂
The tools concerned are the gap-filling methods of
volume.py
, with a closely linked API indDEM
, and several related issues (all pretty old now):DEMCollection
anddDEM
classes #72,Where should we make the distinction?
After many discussions, I see two distinct categories of applicative tools in relation to xDEM:
Tools like 1/ could fit directly in xDEM, because they are generalizable, while 2/ would fit better in a separate repo.
What would this new repo look like?
We also bumped into this question many times in the past years. (coding @adehecq KH9 repo, and others)
If the nature of the data (elevation) is not the center of this new repo (because you have packages like xDEM for different data types), what should it be?
After a lot of thinking and looking around, there seems to be the need for a "statistical estimation" package for EO surface data, which could combine all types of surface data (whether elevation, spectral, velocity, model estimates such as ice thickness, etc).
In the case of volume change, it happens that we "only" need glacier outlines + DEMs. But most of the other variables estimated for glaciers (SMB, frontal ablation, area change, etc) all require data of different nature.
In short, maybe the right way to organize this in a logical unit would be to create a package focused on the statistical estimation for surface applications (focusing mostly on cryo or glaciers?):
There are a lot of actors in the community and small packages that have such tools coded in (like pDEMtools). We could gather all of those in a repo where we can test and document them consistenty, and easily maintain them and build on top. It would also be the perfect occasion to pair with uncertainty propagation software for Xarray (like ObsArray).
What do you think @erikmannerfelt @atedstone @adehecq?
The text was updated successfully, but these errors were encountered: