Skip to content
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

Optimize coregistration module performance for Xarray/Raster objects #543

Open
rhugonnet opened this issue Jun 7, 2024 · 0 comments
Open
Labels
architecture Need to re-organize or re-structure something performance Improvement to feature performance

Comments

@rhugonnet
Copy link
Contributor

We have a lot of steps that extract the NumPy array/transform from a Raster/Xarray dataset, and then re-build the object in the fit function to be able to apply the Raster API consistently. This takes extra memory and computing time.

I see 2 solutions to remedy this issue:

  1. Switch to using Raster and Xarray objects directly in the fit_rst, never going transforming to a NumPy array (just passing the one contained),
  2. Make the Raster API available as separate functions from the class.

We (wisely) decided to not do 1. initially because we did not know if the Raster class was going to be a success. But given that we'll be able to use an Xarray object the same way now, that should not be a problem anymore.
It would still be valuable to have 2. available for some instances.

Minimal changes would be needed for 2., but would make the code harder to read in the Coreg class (calling non-public GeoUtils functions with volatile metadata).

@rhugonnet rhugonnet added performance Improvement to feature performance architecture Need to re-organize or re-structure something labels Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
architecture Need to re-organize or re-structure something performance Improvement to feature performance
Projects
None yet
Development

No branches or pull requests

1 participant