computational-seismology / pypaw

PYthon Process Asdf Wokflow, in short for, pypaw
https://computational-seismology.github.io/pypaw/
GNU Lesser General Public License v3.0
3 stars 6 forks source link

Missing dependencies #17

Open vivek-bala opened 7 years ago

vivek-bala commented 7 years ago

Hey Wenjie,

In step 5 of INSTALL.md, I had to install the following packages explicility for py.test to be successful:

wjlei1990 commented 7 years ago

Hi Vivek,

Thanks for pointing it out. Maybe I am not that clear from the doc...

Did you successfully install pytomo3d? If so, there is also installation doc there. For pyadjoint, we use our own version.

git clone --branch dev https://github.com/chukren/pyadjoint 
vivek-bala commented 7 years ago

Thanks for pointing it out. Maybe I am not that clear from the doc... Did you successfully install pytomo3d? If so, there is also installation doc there

Ah ok, I didn't check that installation doc. Thank you.

For pyadjoint, we use our own version.

Great, I'll use that one then. Thanks.

vivek-bala commented 7 years ago

I'll close this. Not really an issue since I missed part of the documentation.

vivek-bala commented 7 years ago

I still needed to install pyyaml separately. I would suggest adding pyyaml installation to the INSTALL.md file as well.

wjlei1990 commented 7 years ago

fixed. Please pull.

Thanks :)