Opened 11 years ago

Closed 10 years ago

#888 closed enhancement (fixed)

Support RAW photo files

Reported by: Odin Hørthe Omdal (Velmont) Owned by:
Priority: major Milestone: 0.7.0
Component: programming Keywords:
Cc: berkerpeksag, Odin Hørthe Omdal (Velmont) Parent Tickets:

Description

It'd be nice to be able to upload Nikon NEF and Canon CR2 directly to MediaGoblin and let it handle the files.

(Actually, it's even more useful when /not/ uploading but just ingesting local files directly ;))

Attachments (1)

0001-Implement-Raw-Image-media-type-plugin.patch (8.7 KB ) - added by Odin Hørthe Omdal (Velmont) 10 years ago.
Implement Raw Image media type

Download all attachments as: .zip

Change History (8)

comment:1 by berkerpeksag, 11 years ago

Cc: berkerpeksag added

comment:2 by Christopher Allan Webber, 10 years ago

Milestone: 0.7.0
Owner: set to Odin Hørthe Omdal (Velmont)
Status: newin_progress

This looks good, though I have not tried it.

Only a few things:

by Odin Hørthe Omdal (Velmont), 10 years ago

Implement Raw Image media type

comment:3 by Odin Hørthe Omdal (Velmont), 10 years ago

Cc: Odin Hørthe Omdal (Velmont) added

Oh. I never got the email update. Maybe I need to CC myself. :)

I added some siteadmin docs, and rebased the patch to newest master. And I turned off the default enabling, that was not really meant to be there.

It would be very nice if someone would be willing to test it. Did a quick search for a public raw image, here's one for NEF: http://www.imaging-resource.com/PRODS/nikon-d600/YDSC_3937.NEF.HTM -- And here's a (direct link!) for CR2: http://img.photographyblog.com/reviews/canon_eos_5d_mark_iii/sample_images/canon_eos_5d_mark_iii_09.cr2

comment:4 by Odin Hørthe Omdal (Velmont), 10 years ago

Owner: Odin Hørthe Omdal (Velmont) removed
Status: in_progressreview

comment:5 by Christopher Allan Webber, 10 years ago

That's helpful... thanks Velmont :)

I'll try to review this today, but I've also marked this as attached to the 0.7.0 review cycle, so at the very least I'll give it a try before we get ready to release. Thanks!

comment:6 by Christopher Allan Webber, 10 years ago

Owner: set to Christopher Allan Webber
Status: reviewin_progress

I just gave this a test spin. Works great! I've merged it to master.

However, I'm leaving the bug open for a few. There is one problem I want to fix: if you put this plugin *after* the video plugin, the video plugin might intercept those files (which kind of has a goofy result; you get a video player with a still image). I'd like to fix that real quick right before the release.

comment:7 by Christopher Allan Webber, 10 years ago

Owner: Christopher Allan Webber removed
Resolution: fixed
Status: in_progressclosed

Fixed my above complaint. Now the video plugin behaves itself and everyone gets along.

Closing this out. Great work, Velmont!

Note: See TracTickets for help on using tickets.