search

adjtomo / seisflows

An automated workflow tool for full waveform inversion and adjoint tomography
http://seisflows.readthedocs.org
BSD 2-Clause "Simplified" License
183 stars 124 forks source link

pyatoa 0.4.0 cannot be specified. #190

Closed mnagaso closed 9 months ago

mnagaso commented 9 months ago

Hi! I am trying to install Seisflows, but I needed to modify the version of pyatoa to be installed with conda, because of the error below. Do you think this workaround would work? Thank you!


The conflict is caused by:
    The user requested pyatoa>=0.3.0
    seisflows 3.0.0 depends on pyatoa>=0.4.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict
bch0w commented 9 months ago

Thanks for pointing this out @mnagaso, things have gotten a bit out of sync as I have been developing for a research problem. I think this approach will not work as there are some changes in a Pyatoa branch that are required by the current devel branch of SeisFlows. I think the requirement here will be for me to version release Pyatoa v0.4.0. A temporary workaround is to manually install Pyatoa via pip:

conda activate adjtomo  # or environment that contains seisflows
git clone --branch devel git@github.com:adjtomo/pyatoa.git 
cd pyatoa
pip install -e .

I will keep this PR open as I push out the latest Pyatoa version, sorry for the inconvenience!

bch0w commented 9 months ago

That looks great! Thanks @mnagaso

mnagaso commented 9 months ago

Hi Bryant ! (@bch0w)

Thank you very much for your response and explanation. I understood the current situation. If it would be convenient for you, please take my modifications on the setup files👍