search

borevitzlab / spc-eyepi

Raspberry PI DSLR control code
https://borevitzlab.github.io/spc-eyepi
GNU General Public License v3.0
13 stars 3 forks source link

Dcraw version #11

Closed kdm9 closed 9 years ago

kdm9 commented 10 years ago

Is the bundled version of dcraw important? If not, can we just use the debian package?

clixx-io commented 10 years ago

There's nothing special about my compilation of dcraw. So if there is a packaged version of that available there'd be no reason not to use that.

A previously mentioned option was to use a completely pythonic implementation. On 03/07/2014 12:02 pm, "Kevin Murray" notifications@github.com wrote:

Is the bundled version of dcraw important? If not, can we just use the debian package?

— Reply to this email directly or view it on GitHub https://github.com/borevitzlab/spc-eyepi/issues/11.

kdm9 commented 10 years ago

OK, cool. The issue i'd have with the pure python version would be that it's much slower, and it's definitely CPU limited here. I'll advise what to do about this after some consideration, however we can remove dcraw.c (and hence many of the libX-dev compile-time dependecies) from the repo.

Cheers, K

clixx-io commented 10 years ago

Hi Kevin,

btw, when you compiled, did you use the -O4 option ? for speed?

gcc -o dcraw -O4 dcraw.c -lm -ljpeg -llcms2 -DNO_JASPER

On Fri, Jul 4, 2014 at 11:16 AM, Kevin Murray notifications@github.com wrote:

OK, cool. The issue i'd have with the pure python version would be that it's much slower, and it's definitely CPU limited here. I'll advise what to do about this after some consideration, however we can remove dcraw.c (and hence many of the libX-dev compile-time dependecies) from the repo.

Cheers, K

— Reply to this email directly or view it on GitHub https://github.com/borevitzlab/spc-eyepi/issues/11#issuecomment-48000564 .

kdm9 commented 10 years ago

Yeah, I did. If you're happy with the package in debian, I think you can remove dcraw.c and the gcc line from install.sh.

clixx-io commented 10 years ago

I've been testing with the distro version and it hasn't been converting properly.

So I've changed back to the native compiled version.

On Wed, Jul 9, 2014 at 7:08 PM, Kevin Murray notifications@github.com wrote:

Yeah, I did. If you're happy with the package in debian, I think you can remove dcraw.c and the gcc line from install.sh.

— Reply to this email directly or view it on GitHub https://github.com/borevitzlab/spc-eyepi/issues/11#issuecomment-48446342 .