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

wip: openjph HT decoder #1119

Closed
wants to merge 161 commits into from
Closed

Conversation

bradh
Copy link
Contributor

@bradh bradh commented Jan 28, 2024

This is a mostly-complete version of the OpenJPH JPEG-2000 + HT decoder.

To sort out the component to channel mapping, we need to pass down information from the j2kH item property. Is the best (only?) way to do make a C structure on the libheif side and push it down to the decoder plugin (via push_data())?

@farindk
Copy link
Contributor

farindk commented Jun 25, 2024

To sort out the component to channel mapping, we need to pass down information from the j2kH item property. Is the best (only?) way to do make a C structure on the libheif side and push it down to the decoder plugin (via push_data())?

The best way is probably to extend the plugin API with a J2K specific decoding function or a function to set the mapping.
Probably it will look something like this:

void (*set_jpeg2000_channel_mapping)(void* decoder, const int* channel, const int* component, int nComponents);

I don't know what is needed specifically, please specify as needed.

bradh and others added 28 commits July 10, 2024 11:31
…7-10

compression: add (internal) api documentation for compression
modify no-colons option only replace in auxType
uncompressed: fix box dump when profile is 0
fix heif_dec no-colons option for windows
Fix libbrotlidec pkg-config requires
tiff: Fix wrong type for "seekTIFF" function.
- Run "cmake" before building.
- Enable more codecs.
- Build on PRs (but don't upload to Coverity).
…rty-build-errors

CI: Fail if third-party build fails.
rename property raw data implementation to match header
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.