adjtomo/seisflows

Issue with the updated seisflows installation

Closed this issue · 3 comments

I have been trying to freshly install seisflows and pyatoa under an environment called 'adjtomo'. However, I noticed that the seisflows package does not require the requirements.txt file anymore. On following the instructions (https://seisflows.readthedocs.io/en/latest/), it seems to create a new environment called 'seisflows'.
How do I fix this?
Selection_076
Selection_075
Selection_074

bch0w commented

Hi @mishuyetsam, sorry you're having trouble with installation. Yes, since the changes introduced in #149, we did away with the requirements.txt and setup.py files in favor of a new pyproject.toml file. These accomplish the same task of allowing SeisFlows to be installed as a package.

Thank you for pointing this out. After trying this out myself I realize the alternative instructions are incorrect and will only allow creating an environment called seisflows as you have experienced. To update an existing environment as you would like, you will need to add a -n/--name flag to specifically name the Conda environment you want to install to. That is:

conda create -n adjtomo
# git clone and cd to seisflows
conda env update -f environment.yml -n adjtomo
conda activate adjtomo

I just tried this out on my machine and was successfully able to install SeisFlows into my adjtomo environment

(adjtomo) [bchow@blackbox seisflows]$ conda env update -f environment.yml -n adjtomo
...
# To activate this environment, use
#
#     $ conda activate adjtomo
#
# To deactivate an active environment, use
#
#     $ conda deactivate
(adjtomo) [bchow@blackbox seisflows]$ seisflows
usage: seisflows [-h] [-w [WORKDIR]] [-p [PARAMETER_FILE]]
                 {setup,configure,swap,submit,resume,restart,clean,par,sempar,check,init,plot2d,plotst,print,reset,debug,examples}

Please let me know if that works for you. If so then I will update the documentation page with the corrected installation procedure.

Thank you for the concise explanation. I followed it and it's finally set up and running!

bch0w commented

Closing as complete! Please re-open if this issue still persists for you.