search

megbedell / wobble

precise data-driven RV fitting, now with tellurics!
https://wobble.readthedocs.io
MIT License
39 stars 16 forks source link

needs requirements list #55

Closed megbedell closed 5 years ago

megbedell commented 5 years ago

Some users have c++ compiling issues (for the interp op) when attempting to install. We should figure out what the requirements are and make them explicit. @dfm @rodluger help??

dfm commented 5 years ago

From the C++ side, the only thing that is required is tensorflow.

megbedell commented 5 years ago

ok - more details on the C++ issues in #57

bjfultn commented 5 years ago

Yeah, I found this to be a macOS Mojave issue rather than a Cython issue as I originally suspected. Still a requirements file is needed. Starting with tensorflow

dfm commented 5 years ago

Good call!

Yeah - there's no Cython in the house here :-)

Our method for building TensorFlow extensions is a bit of a hack but it's much more sustainable (for us) than the method recommended by TensorFlow crew.