one trivial step missing in the installation

Issue #185 resolved
Former user created an issue

There is no mention about the step 'python setup.py install --user'

Comments (4)

  1. Jiamin Wang

    if you install Amp using 'python setup.py install --user', Amp might not use fortran modules, which make the regression much slower.

  2. andrew_peterson repo owner

    If I understand correctly, setup.py does not currently compile the fortran modules. We should try to get it to do that.

  3. Efrem Braun

    By the way, I realize that my merge request #22 should actually be corrected. I was mistaken in thinking that python setup.py install user was needed for the installation. In fact, either doing that or adding ASE to the pythonpath environment variable is enough on its own. The documentation should probably be changed to be similar to what's on https://wiki.fysik.dtu.dk/ase/install.html#installation-from-source. I can either put in an easy pull request for this now or wait for you guys to figure out what you want to do with the fortran modules. Sorry for the error!

  4. Muammar El Khatib

    setup.py can build fortran modules.

    In this commit setup.py has been modified so that 'python setup.py install --user' can be used to install Amp. This fixes #185.

    → <<cset c62c1d14db42>>

  5. Log in to comment