search

DedalusProject / dedalus

A flexible framework for solving PDEs with modern spectral methods.
http://dedalus-project.org/
GNU General Public License v3.0
489 stars 115 forks source link

Installation currently requires Cython<3 #266

Closed geoffstanley closed 1 year ago

geoffstanley commented 1 year ago

Trying to install Dedalus3 following the recommended instructions worked for me in the past, but just failed with a Getting requirements to build wheel did not run successfully. error, including a lot of Error compiling Cython file lines.

Following this, I got Dedalus3 re-installed by adding a Cython<3 requirement to the pip3 install command from the recommended instructions:

echo "Cython<3" > cython_constraint.txt
CC=mpicc PIP_CONSTRAINT=cython_constraint.txt pip3 install --no-cache http://github.com/dedalusproject/dedalus/zipball/master/

Just noting this in case anyone else has trouble, or perhaps the recommended instructions should be updated to reflect this. I guess it's just temporary though until Dedalus3 gets working with Cython 3.

kburns commented 1 year ago

Thanks, yeah we have a similar fix to pin cython via conda in the testing pipeline, and it looks like it's enough for now.

geoffstanley commented 1 year ago

I see you've updated the installation instructions and build requirements. Perfect, thanks!