-
Notifications
You must be signed in to change notification settings - Fork 8
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
RAW file support #34
Comments
I'd like to take a stab at working on this. My approach would be to implement a The output would be similar to ffmpeg - thumbnail and image previews. A secondary addition (not necessarily tied to raw images) would be to configure exporting/download images - eg. "Download as RAW", "Download as JPG", "Download as ... (configure)" |
Hey, that would be great, thanks! I experimented a bit with imagemagick as it seems to support a lot of RAW files as well (as noted by @DawidPietrykowski in #55 (comment)). As a bad developer I didn't integrate it into master and only pushed it as a branch now, sorry about that 🙈. The reason is that it works "ok", but it's quite heavy/slow, so probably not a great experience. In any case, you have the right idea on how to add it and perhaps the imagemagick commit above can serve as a further example. I'd be most interested in knowing how it behaves in a "real-time" scenario, i.e. you have a view with 10 photos, so On the secondary download button idea, it's a bullet point in #26. I implemented a crude version at some point, but I didn't like how it looked, so I left it in, but commented out (great I know!). It only exposes all the sources right now, but with some extension it might be useful as a general "export" function too. Let me know if you need some support on the implementation and I'll try to help! |
Oh awesome, thank you for all that info. That would actually help quite a bit. Darktable-cli supports multiple For me, with highquality resampling, it takes 4-5 seconds per raw CR2 and CR3 photo (~28MB-32MB files) to convert them on an i5-8350U laptop CPU. Darktable takes about 1-2 seconds to start before converting. Without highquality resampling, it takes half the time. Exporting only Image Magick support is something I'm interested in as well for providing previews to Photoshop's PSD files. |
Good findings, thanks! So it's not super fast, but also not too bad. Would still be interesting to try it out as an on-the-fly decoder. I wonder if you can make it spit out a photo as a "raw" (for the lack of a better term) RGB image. While that's not practical for saving the file, my guess is that it'll be faster to read/copy that in as a Go image as you avoid the jpeg encode and decode step. That's what the ffmpeg/imagemagick implementations do via PAM files, which are just RGB with a text header. Similarly you might want to avoid decoding to the original resolution if it's not needed (loading an 8k photo to show on a 720p phone is a waste), check the ffmpeg/imagemagick configs on how to set up the different scales. I haven't tried the imagemagick branch on PSDs, let me know if you give it a spin. And again, happy to support in case of questions! 😄 |
Yeah, a lot of my raw photos are typically coupled with jpgs anyway (usually shoot in both raw and jpg since it makes previews much quicker and it's easy to send off "proof" photos quickly), so dark-table would be more on-the-fly and for raws that don't have an accompanying jpg. The above use case actually brings up the question (which I guess loops back to the "Download as" option in #26), where it may be worthwhile to have an option to group similar-titled images and assume they're related (eg. "IMG_0001.JPG" is related to "IMG_0001.RAW", so we can show the raw as a download option). I didn't see a way to export from darktable other than jpg/png, but there's bound to be something there. I'll try implementing this within the next few days (if I don't get side tracked playing around with other features...) |
Yeah grouping those is a good idea, though personally I don't have many cases for that right now. I have been thinking of a deduplication feature, it could be related to that as well. Dedup to not show the same photo multiple times + include related media in the download section. Just FYI I might be a bit less responsive in the next 1-2 weeks, but I'll try my best :) |
Hi @ckyb! Just following up, did you get the chance to look into this? :) I can help you get set up if you'd like. |
Hey Miha, I'm finally allocating some free time for development. I'll push up a fork tracking the work when I get started, and hopefully open up a PR for review. |
Hey there! That's great to hear! Just as a heads up, I'll be a bit busy for the next month or so, so I might not be able to respond quickly. Feel free to join the Discord if you'd like a more freeform chat that you don't want to do over GH issues as well. As for RAW support, you could look into supporting that as a "variant" of the jpeg. For example, all thumbnails are just considered variants of the same photo and they are used for rendering, but otherwise don't really show up as duplicates anywhere. The same goes for the different video variants that Synology Moments creates (eg an h264 version) that are used as fallbacks in case the original doesn't play well in the browser. All of the variants should also already be available in the right-click context menu, so if you add RAW as a variant, even if the data itself is not used anywhere, it would be an easy way to make it downloadable. I also tried once to add a more explicit download button to the fullscreen photo view and there might be commented out code there to this effect, but it didn't look right at the time. You could revive that as well if you're interested :) |
Is your feature request related to a problem? Please describe.
RAW files are completely unsupported right now so it's not possible to see them at all.
Describe the solution you'd like
Transparent support for common formats so that they just look like any other photo. Depending on how long it takes to convert, this might need thumbnail generation (issue TBD). I believe often they are also paired with JPEGs, which would be nice to take into account.
Additional context
Can be a deal breaker for many if not supported, see H_Q_'s comment
The text was updated successfully, but these errors were encountered: