-
Notifications
You must be signed in to change notification settings - Fork 11
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
Support png bit depths 1, 2 and 4 #526
base: develop
Are you sure you want to change the base?
Support png bit depths 1, 2 and 4 #526
Conversation
I think you will get the missing test runs when you merge in the develp branch. These look like good changes, and thanks a lot for the tests! Let me know when this is ready to merge. |
@webisu Do you want to test this branch with wgrib2 to see if png with 1, 2 and 4 bit depths works as expected? |
Wgrib2 doesn't use g2c right now.
…On Tue, Oct 8, 2024 at 11:51 AM Dusan Jovic ***@***.***> wrote:
@webisu <https://github.com/webisu> Do you want to test this branch with
wgrib2 to see if png with 1, 2 and 4 bit depths works as expected?
—
Reply to this email directly, view it on GitHub
<#526 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEIB7ZSW3TMIVAXNBHR7AVTZ2P5PRAVCNFSM6AAAAABPKJSUD2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGIZDCNBSG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
However, wgrib2 is about to start using g2c for compression. @DusanJovic-NOAA I suggest that if this passes your tests we merge it and it can be tested further with wgrib2 later. |
…cmake.in When this library is compiled as a submodule of another project the location of PackageConfig.cmake.in template file must be specifield using this project's source directory (PROJECT_SOURCE_DIR) not the parent project's (top-level) source directory (CMAKE_SOURCE_DIR)
Update png pack/unpack and encode/decode routines to support bit depths 1, 2 and 4.
Fixes #508