mjuric / conda-lsst

Building Conda packages for the LSST stack
4 stars 8 forks source link

Enable building against MKL-enabled Anaconda #71

Closed mjuric closed 8 years ago

mjuric commented 8 years ago

This is fundamentally an awful hack, but it makes it possible to build conda packages that work with MKL versions of numpy, scipy, etc. We:

Note: I worry that our issues with MKL could be indicative of a real problem on our end. Our code doesn't crash, it just returns (very) incorrect results.

It may be that we're just not checking the return codes from FFTW3 calls and detecting when they fail (MKL doesn't implement everything FFTW3 proper does), but it's also possible we're (incorrectly) assuming something about FFTW3's memory management that just happens to work with the reference implementation but not MKL wrappers.

This could be worth understanding; the number of places where we're using FFTW isn't that large (I think it's only for sinc interpolation), so it may not be very difficult.

mjuric commented 8 years ago

A better way to handle this has been put forward as a part of #73.