Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with the updated seisflows installation #166

Closed
Mishrasamriddhi opened this issue Mar 27, 2023 · 3 comments
Closed

Issue with the updated seisflows installation #166

Mishrasamriddhi opened this issue Mar 27, 2023 · 3 comments

Comments

@Mishrasamriddhi
Copy link

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
Copy link
Member

bch0w commented Mar 28, 2023

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.

@Mishrasamriddhi
Copy link
Author

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

@bch0w
Copy link
Member

bch0w commented Jul 12, 2023

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

@bch0w bch0w closed this as completed Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants