powellb / seapy

State Estimation and Analysis in Python
MIT License
28 stars 21 forks source link

No module named 'seapy.external' #65

Closed huyquangtranaus closed 3 years ago

huyquangtranaus commented 3 years ago

Hi,

Is there any suggestion to fix the following error message:

"/home/hugo/Pyroms_roms/seapy/seapy/oa.py", line 13, in from seapy.external import oalib ModuleNotFoundError: No module named 'seapy.external'

Thanks Huy

huyquangtranaus commented 3 years ago

I answer myself:

I reinstalled everything using the following approach:

git clone https://github.com/powellb/seapy.git pip install -e seapy

Then it works. Thanks

powellb commented 3 years ago

As you note, the original problem is that you had not installed the package. In your second message, you are actually doing two separate things:

The git clone step provides a copy of the code on your machine. To install the git version, you simply need to:

git clone https://github.com/powellb/seapy.git
cd seapy
python setup.py install

You instead installed via pip install -e seapy, this downloads a package and installs it. So, you don’t need the git clone if you install via pip.

huyquangtranaus commented 3 years ago

Right, so should this Readme.md be updated? Screenshot from 2021-10-29 20-04-35

powellb commented 3 years ago

Good point. Thanks!

On Oct 28, 2021, at 11:07 PM, Huy Quang Tran @.***> wrote:

 Right, so should this Readme.md be updated?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

powellb commented 3 years ago

Actually (I didn't write the Readme), what you did (and the Readme) are correct. The pip install -e option uses the local copy you downloaded...

On Oct 28, 2021, at 11:07 PM, Huy Quang Tran @.***> wrote:

Right, so should this Readme.md be updated? https://user-images.githubusercontent.com/32722297/139408241-71a7423c-493c-4c8a-9ee6-9dc6fca07ecd.png — You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/powellb/seapy/issues/65#issuecomment-954578533, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAANGP2XWHTSCEU7SMLMCVDUJJP6ZANCNFSM5G4DIJXA. Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.