Skip to content

Jpeg2000#154

Closed
maforget wants to merge 2 commits intomasterfrom
Jpeg2000
Closed

Jpeg2000#154
maforget wants to merge 2 commits intomasterfrom
Jpeg2000

Conversation

@maforget
Copy link
Copy Markdown
Owner

@maforget maforget commented Mar 6, 2025

Added support for reading Jpeg2000 files (.jp2 & .j2k). Write support is possible but the current library used seems bugged and returns the wrong gamma, so I've removed it.

A known issue to disabling write support like that, is when exporting a Jpeg2000 file that had pages merged and keeping the setting to Preserve Original, that merged page will be created as a normal Jpeg, but keep it's .jp2 extension. Seem a very very niche problem, not sure if it warrants checking into it more.

maforget added 2 commits March 6, 2025 11:53
Because image is bugged with an incorrect Gamma. Don't think it matters so will just disable it.

cureos/csj2k#11
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 6, 2025

Build Successful! You can find a link to the downloadable artifact below.

Name Link
Commit c012f26
Logs https://github.com/maforget/ComicRackCE/actions/runs/13704045419
Download https://nightly.link/maforget/ComicRackCE/suites/35308348649/artifacts/2705281377

maforget added a commit that referenced this pull request Apr 5, 2025
commit c012f26
Author: maforget <11904426+maforget@users.noreply.github.com>
Date:   Thu Mar 6 03:30:21 2025 -0500

    Removed export of Jpeg2000 images

    Because image is bugged with an incorrect Gamma. Don't think it matters so will just disable it.

    cureos/csj2k#11

commit 41b0ae7
Author: maforget <11904426+maforget@users.noreply.github.com>
Date:   Thu Mar 6 02:43:14 2025 -0500

    Added Support for Jpeg 2000 files (.jp2 or .j2k)
@maforget maforget closed this Apr 5, 2025
@maforget maforget deleted the Jpeg2000 branch April 5, 2025 22:31
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.

1 participant