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

No primal feasible solution #4

Closed
tminka opened this issue Aug 8, 2020 · 6 comments
Closed

No primal feasible solution #4

tminka opened this issue Aug 8, 2020 · 6 comments

Comments

@tminka
Copy link

tminka commented Aug 8, 2020

When running the commands in the README, glpsol -m OSeMOSYS.txt -d simplicity.txt says there is no primal feasible solution. This is due to the numbers in the data/default_values.csv file. For example, ModelPeriodEmissionLimit,-1

@willu47
Copy link
Member

willu47 commented Aug 11, 2020

Hi @tminka - thanks for the report. Which version of the OSeMOSYS code are you using? Default values of -1 were introduced under OSeMOSYS/OSeMOSYS_GNU_MathProg#30 to fix issues with upper bounds of 0 being ignore for certain constraint parameters (OSeMOSYS/OSeMOSYS_GNU_MathProg#13 and OSeMOSYS/OSeMOSYS_GNU_MathProg#26).

You may wish to try a more recent version of the OSeMOSYS GNU MathProg code with this version of the simplicity model.

@willu47 willu47 closed this as completed Aug 11, 2020
@tminka
Copy link
Author

tminka commented Aug 11, 2020

I am using the version of the OSeMOSYS code that I get by default when I clone the OSeMOSYS repository (with submodules). It is at https://github.com/OSeMOSYS/OSeMOSYS_GNU_MathProg/blob/5daea546643298be9d84b0108e4c2dd8e228ee7b/src/osemosys.txt

@willu47
Copy link
Member

willu47 commented Aug 11, 2020

Hi @tminka - understood. We haven't yet established a good release system for the whole OSeMOSYS family yet, and so that version of the GNU MathProg code is still rather old.

Please update to the newest version of the GNU MathProg code by following these steps from the root of the OSeMOSYS repository:

cd OSeMOSYS_GNU_MathProg
git checkout master
git pull

This should update the repository for OSeMOSYS_GNU_MathProg with the latest version of the code.

@tminka
Copy link
Author

tminka commented Aug 11, 2020

Unfortunately the newest version of the GNU MathProg code doesn't work with simplicity since simplicity does not define DiscountRateStorage. The Training_Case_Studies (atlantis and utopia) have the same problem.

@willu47
Copy link
Member

willu47 commented Aug 11, 2020

Very true - then it is a simple matter to git checkout xxxxxx the version of the OSeMOSYS code which does work with simplicity... 1567dbe2d341841c97e6fdb9fa130bd22d66fe83 should be fine.

So try git checkout 1567dbe2d.

It's clear we should add better documentation to the simplicity model so that it is less difficult to find a version of the model it is compatible with!

@tminka
Copy link
Author

tminka commented Aug 11, 2020

Thanks, that works. I will create a PR to update the documentation.

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

Successfully merging a pull request may close this issue.

2 participants