Skip to content

Support Hyperspectral Imagery#1660

Draft
dukesook wants to merge 7 commits intostrukturag:masterfrom
dukesook:hyperspectral
Draft

Support Hyperspectral Imagery#1660
dukesook wants to merge 7 commits intostrukturag:masterfrom
dukesook:hyperspectral

Conversation

@dukesook
Copy link
Contributor

@dukesook dukesook commented Jan 5, 2026

Refactors the m_planes variable from an std::map to an std::vector to support multiple components per channel.

@farindk
Copy link
Contributor

farindk commented Jan 31, 2026

Thanks, Devon. This is still marked as a draft. I can take over from here if you want.

@dukesook
Copy link
Contributor Author

dukesook commented Feb 1, 2026

Yes, please take it over as this is a very important capability. I apologize for not finishing, but I got pulled into doing another task.

I imagine you'll start from a fresh branch as this PR got messy real quick. There wasn't really an incremental way of implementing.

@farindk
Copy link
Contributor

farindk commented Feb 3, 2026

I imagine you'll start from a fresh branch as this PR got messy real quick. There wasn't really an incremental way of implementing.

What was the main hurdle that made this difficult? On first glance, the approach in your PR looks reasonable and I would have done it in a similar way. From your experience, what should I try to do different?

@dukesook
Copy link
Contributor Author

dukesook commented Feb 4, 2026

I feel good in about the overall direction of my approach. In fact, used this branch to demonstrate converting a 16-bit hyperspectral image to HEIF.

However, there are many opportunities for bugs and I would have preferred to implement it incrementally somehow.

@farindk
Copy link
Contributor

farindk commented Feb 4, 2026

However, there are many opportunities for bugs and I would have preferred to implement it incrementally somehow.

Yes, I understand that. It will require changes and review throughout the code...
Not a change to carry out on the master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants