Skip to content

Conversation

alinelena
Copy link
Member

No description provided.

Copy link
Member

@ElliottKasoar ElliottKasoar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally looks good, thanks!

We should add a test for the hdf5 bands, and it looks like there are a couple of LZMAErrors in the tests still.

@ElliottKasoar ElliottKasoar added the enhancement New/improved feature or request label Jul 24, 2025
oerc0122
oerc0122 previously approved these changes Aug 12, 2025
Copy link
Collaborator

@oerc0122 oerc0122 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks ok codewise when it works, just one small potential clarification.

alinelena and others added 6 commits September 29, 2025 16:18
Co-authored-by: Elliott Kasoar <45317199+ElliottKasoar@users.noreply.github.com>
Co-authored-by: Elliott Kasoar <45317199+ElliottKasoar@users.noreply.github.com>
Co-authored-by: Elliott Kasoar <45317199+ElliottKasoar@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New/improved feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants