Skip to content

Add test to validate python module reading of unfinished parts#2421

Merged
cary-ilm merged 2 commits into
AcademySoftwareFoundation:mainfrom
cary-ilm:unfinished-multipart-test
May 18, 2026
Merged

Add test to validate python module reading of unfinished parts#2421
cary-ilm merged 2 commits into
AcademySoftwareFoundation:mainfrom
cary-ilm:unfinished-multipart-test

Conversation

@cary-ilm

Copy link
Copy Markdown
Member

PR #2148 extends the python module to skip over parts with invalid/unfinished pixel data, so it reads what it can. Reading "header_only" reads all headers.

This adds a test to validate that behavior, using a test file with corrupted chunk tables in two of the parts.

Made-with: Cursor

cary-ilm added 2 commits May 16, 2026 11:34
PR AcademySoftwareFoundation#2148 extends the python module to skip over parts with
invalid/unfinished pixel data, so it reads what it can. Reading
"header_only" reads all headers.

This adds a test to validate that behavior, using a test file with
corrupted chunk tables in two of the parts.

Made-with: Cursor
Signed-off-by: Cary Phillips <cary@ilm.com>
@cary-ilm cary-ilm merged commit 99d87b4 into AcademySoftwareFoundation:main May 18, 2026
9 of 10 checks passed
cary-ilm added a commit that referenced this pull request May 21, 2026
PR #2148 extends the python module to skip over parts with
invalid/unfinished pixel data, so it reads what it can. Reading
"header_only" reads all headers.

This adds a test to validate that behavior, using a test file with
corrupted chunk tables in two of the parts.

Made-with: Cursor

Signed-off-by: Cary Phillips <cary@ilm.com>
palemieux pushed a commit to sandflow/openexr-ht that referenced this pull request Jun 10, 2026
…mySoftwareFoundation#2421)

PR AcademySoftwareFoundation#2148 extends the python module to skip over parts with
invalid/unfinished pixel data, so it reads what it can. Reading
"header_only" reads all headers.

This adds a test to validate that behavior, using a test file with
corrupted chunk tables in two of the parts.

Made-with: Cursor

Signed-off-by: Cary Phillips <cary@ilm.com>
Signed-off-by: Pierre-Anthony Lemieux <pal@sandflow.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant