-
Notifications
You must be signed in to change notification settings - Fork 288
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
ISO Media / ISO/IEC 14496-12 support. #235
Comments
Roman: Good to hear from you. I hope everything's good with you. I replied to #236 without realising that you initiated that report. |
I think I've got a good understanding about the scope of this project and documented my investigation #236 We're having a team meeting in May #225 when we'll discuss the specification and schedule for v0.27. Read/write support for WebP was added for v0.26 in August/September 2016 and took about 4 weeks for two engineers (that's totally finished with test harness and building on all platforms). |
Hopefully the same mistake won't be repeated yet over again, and there won't be a write support for CR3. |
I'm not sure I've understood what you've said. We support read/write on most formats. Why would we not support read/write for CR3? Even to change the timestamp in an image requires read-write support. If you're wondering why WebP took 4 weeks, it's because it's 1000 lines of C++, plus test and build support. Neither Ben nor I were working full-time. Perhaps about 100 hours each over about 1 month. So, the effort required to implement CR3 is probably 100-200 hours. On the subject of changing the timestamps in a file, it would be possible to implement a much faster method than the current code. A user in Canada has implemented his own stand-alone program to achieve this. The feature - turbo update time-stamps - is below the horizon. By that I mean that I have thought about it and discussed it with a user. However it's only "an idea" at the moment. |
That was exactly the point. But i'm going to stop with my offtopic/hijacking comments now, sorry. |
Ah, yes. I now understand your point of view. We could declare CR3 as readonly because editing the image makes it no longer RAW. You've reduced the effort of this project by 70%! Exiv2 knows nothing about pixels. It only reads/write/modifies metadata. Image data (pixels) are preserved. Exiv2 should enable CR3 read/write for time-stamps and other non-image metadata. Incidentally, in Googling into CR3, some folks believe CR3 = Compressed Raw 3. I'm not convinced. I think CR3 = Canon Raw 3. However, I suspect it's either 14 bit colour or uses lossless compression. However it isn't RAW at 14 bits/channel. |
https://bugs.kde.org/show_bug.cgi?id=387351#c1 (that whole bug, really) |
Shipped in v0.27.4 RC1. |
(Cross-reference with darktable-org/rawspeed#121)
Required for #236
The text was updated successfully, but these errors were encountered: