Skip to content

Add Canon EOS Rebel SL3, a regional alias to the 250D#4

Open
xfalcox wants to merge 25 commits intocytrinox:canon_cr3from
xfalcox:canon_cr3
Open

Add Canon EOS Rebel SL3, a regional alias to the 250D#4
xfalcox wants to merge 25 commits intocytrinox:canon_cr3from
xfalcox:canon_cr3

Conversation

@xfalcox
Copy link
Copy Markdown

@xfalcox xfalcox commented Sep 10, 2021

Tested this change locally with a photo from my SL3.

cytrinox and others added 23 commits September 10, 2021 20:46
This patch is based on a draft from Roman Lebedev.
This decompressor is ported from libraw. Some of the file handling logic
has been replaced by rawspeed::Buffer and derived classes.
Most of the code is untouched.
This decoder contains the logic to parse a CR3 ISOBMFF file.
IAD1 contains 4 definitions for rectangles:
 * crop rectangle
 * left optical black area rectangle
 * top optical black area rectangle
 * active area rectangle

This patch parses all IAD1 rectangles and uses them for
black areas and crop.
 * Canon EOS 250D
 * Canon EOS M6 Mark II
 * Canon EOS M200
 * Canon EOS M50 Mark II
 * Canon PowerShot G5 X Mark II
 * Canon PowerShot G7 X Mark III
@cytrinox
Copy link
Copy Markdown
Owner

Thanks for reporting the alias. Would you mind rebasing your branch? I've updated my canon_cr3 yet to match the newest rawspeed development branch.

@xfalcox
Copy link
Copy Markdown
Author

xfalcox commented Sep 14, 2021

Rebased @cytrinox

@xfalcox
Copy link
Copy Markdown
Author

xfalcox commented Oct 28, 2021

Ping @cytrinox

@cytrinox cytrinox force-pushed the canon_cr3 branch 3 times, most recently from 613dd16 to 09ca1df Compare February 15, 2022 17:10
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.

4 participants