Skip to content

OpenJPEG: disable "TPSOT fix" with default STRICT=YES open option#11270

Closed
rouault wants to merge 2 commits intoOSGeo:masterfrom
rouault:openjpeg_TPSOT_COMPLIANT
Closed

OpenJPEG: disable "TPSOT fix" with default STRICT=YES open option#11270
rouault wants to merge 2 commits intoOSGeo:masterfrom
rouault:openjpeg_TPSOT_COMPLIANT

Conversation

@rouault
Copy link
Copy Markdown
Member

@rouault rouault commented Nov 13, 2024

Fixes the issue reported in https://lists.osgeo.org/pipermail/gdal-dev/2024-November/059805.html Requires openjpeg >= uclouvain/openjpeg#1560 (likely 2.5.3 when it is released)

@rouault rouault added this to the 3.11.0 milestone Nov 13, 2024
@rouault rouault force-pushed the openjpeg_TPSOT_COMPLIANT branch from cf0260e to 76a238c Compare November 13, 2024 16:57
@rouault rouault changed the title OpenJPEG: add a TPSOT_COMPLIANT open option that defaults to YES OpenJPEG: disable "TPSOT fix" with default STRICT=YES open option Nov 13, 2024
@rouault
Copy link
Copy Markdown
Member Author

rouault commented Nov 13, 2024

@kidanger I've just ammended this PR to actually fold that new TPSOT_COMPLIANT open into the existing STRICT one. So the default STRICT=YES open option will disable the TPSOT fixing heuristics

@rouault
Copy link
Copy Markdown
Member Author

rouault commented Nov 13, 2024

Dropping this PR as it is no longer needed, since I've actually tuned the openjpeg one to also disable the TPSOT fixing logic, when opj_decoder_set_strict_mode(, true) is called.

@rouault rouault closed this Nov 13, 2024
@coveralls
Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 73.702% (-0.002%) from 73.704%
when pulling 86a7c68 on rouault:openjpeg_TPSOT_COMPLIANT
into a0f5127 on OSGeo:master.

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