Skip to content
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

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

Open
wants to merge 25 commits into
base: canon_cr3
Choose a base branch
from

Conversation

xfalcox
Copy link

@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
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
Author

xfalcox commented Sep 14, 2021

Rebased @cytrinox

@xfalcox
Copy link
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