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

AttributeError: module 'train' has no attribute '__path__' #27

Closed
patterntrade opened this issue Jun 6, 2019 · 3 comments
Closed

AttributeError: module 'train' has no attribute '__path__' #27

patterntrade opened this issue Jun 6, 2019 · 3 comments

Comments

@patterntrade
Copy link

patterntrade commented Jun 6, 2019

Baselines Zoo

Following the example scripts on the GitHub welcome page.

./run_docker_cpu.sh python -m train.py --algo ppo2 --env MountainCar-v0 -n 50000 -optimize --n-trials 1000 --n-jobs 2 \   --sampler random --pruner median

python -m train.py --algo ppo2 --env MountainCar-v0 -n 50000 -optimize --n-trials 1000 --n-jobs 2 \   --sampler random --pruner median

Both give the same error msg:

... Error while finding module specification for 'train.py' (AttributeError: module 'train' has no attribute '__path__') #27

System Info

  • Mac 10.14.4 (Mojave)
  • Python 3.6.8

Additional context
Other scripts, like
./run_docker_cpu.sh python train.py --algo ppo2 --env CartPole-v1
run OK.

@araffin
Copy link
Owner

araffin commented Jun 6, 2019

Hello,

There seems to be a typo in the readme. This should be python train.py (without the -m)

@patterntrade patterntrade changed the title Error while finding module specification for 'train.py' (AttributeError: module 'train' has no attribute '__path__') AttributeError: module 'train' has no attribute '__path__' Jun 6, 2019
@patterntrade
Copy link
Author

Error stemming from writing

... python -m train.py ...

Instead of just

... python -m train ...

Perhaps the documentation could be updated accordingly?

Thanks for a GREAT code project in constant development, especially appreciating the hyper parameter tuning now included.

@araffin
Copy link
Owner

araffin commented Jun 6, 2019

Perhaps the documentation could be updated accordingly?

Yep, will be fixed in the next PR (HER-support)

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

No branches or pull requests

2 participants