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

Installation / Documentation mismatch #8

Open
AtomicAntimatter opened this issue Sep 19, 2017 · 2 comments
Open

Installation / Documentation mismatch #8

AtomicAntimatter opened this issue Sep 19, 2017 · 2 comments

Comments

@AtomicAntimatter
Copy link

issue is just capitalization of module name (Monsoon vs monsoon)

after installing monsoon through
pip install monsoon

to import, it had to be
import Monsoon

Documentation refers to the lowercase import which will fail

 import monsoon.LVPM as LVPM
 import monsoon.sampleEngine as sampleEngine
 import monsoon.Operations as op
@luiscruz
Copy link

luiscruz commented Oct 25, 2017

I'd suggest changing the module name to lowercase 'monsoon', according to Python style guide.

@MaxwellVolz
Copy link

Came across this issue today. Still relevant.

It appears that there may be a discrepancy between the package installation directory and the recommended import statement in the package's documentation.

To complete Sampling from the main channel from Power Monitor Python API.pdf:

import Monsoon.LVPM as LVPM
import Monsoon.sampleEngine as sampleEngine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants