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

Cannot import Amply from pulp #9

Closed
H3yfinn opened this issue Mar 14, 2023 · 2 comments
Closed

Cannot import Amply from pulp #9

H3yfinn opened this issue Mar 14, 2023 · 2 comments

Comments

@H3yfinn
Copy link

H3yfinn commented Mar 14, 2023

Putting this here because it only occured because i was trying to use simplicity. It doesnt happen in our standard use of Otoole/Osemosys.

It's something i noticed while trying to run: otoole convert datapackage datafile simplicity/OSeMOSYS-simplicity-11a3a26/datapackage.json ./simplicity.txt
Error: Cannot import Amply from pulp

env\Lib\site-packages\otoole\preprocess\datafile_to_datapackage.py
Fix:
-installed amply using pip install amply
-changed datafile_to_datapackage.py so it does: import amply as Amply
-It works.

Also just to note, simplicity doesnt work with Otoole v1 because it cannot do the datapackage conversion anymore.

@H3yfinn
Copy link
Author

H3yfinn commented Mar 14, 2023

I see the recent update. Just another note, the DiscountRateStorage csv is not in the dataset that is downloaded via wget. I will close this comment tomorrow.
EDIT: i see the admin for this repo can hide this comment. I'll leave that to them then (:

@willu47
Copy link
Member

willu47 commented Mar 15, 2023

Thanks for the report @H3yfinn. I've now created the new version of simplicity v1.0 and updated the readme. Let me know if you have any other problems.

@willu47 willu47 closed this as completed May 12, 2023
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

2 participants