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

Model file used to run with the python script needs to be edited to run #53

Open
tniet opened this issue Apr 27, 2021 · 4 comments
Open

Comments

@tniet
Copy link
Contributor

tniet commented Apr 27, 2021

The model file needed to run OSeMOSYS global is not the same as the fast code at https://github.com/OSeMOSYS/OSeMOSYS_GNU_MathProg/tree/master/src. Do we need to advocate for an additional model file in the OSeMOSYS repo that is setup for the python script or for the OSeMOSYS fast code to be changed to be python script compatible?
My initial view is that we should advocate for the fast code to be migrated to run by default with the script - this is the fastest way to run the model, and the fast code should be the fastest way to run the model.
Other thoughts? If we have a preferred structure we can ask for this on the OSeMOSYS repo.

@willu47
Copy link
Member

willu47 commented Apr 27, 2021

Ideally, you'd use a specific release of the OSeMOSYS code (e.g. downloaded from Github releases on OSeMOSYS_GNU_MathProg).

@tniet
Copy link
Contributor Author

tniet commented Apr 27, 2021

Yes @willu47, but the fastest way to run the model isn't with a released version - maybe (per your note at OSeMOSYS/OSeMOSYS_GNU_MathProg#66 (comment)), we need to consider what the fast code is meant to be? I would argue that it should be the fastest way to run models (I'll comment on the MathProg issue as well).

On a related note, I wonder if @trevorb1 can run with the fast and the script and let us know the difference in processing times?

@tniet
Copy link
Contributor Author

tniet commented Apr 27, 2021

Hey @willu47 - yes, ideally we'd use a release version. However, that's not the fastest way to run the model as GNU MathProg isn't very efficient at creating the MODEx crossover sets. This is related to the issue at OSeMOSYS/OSeMOSYS_GNU_MathProg#66 that you posted and I will comment further there.

Maybe @trevorb1 can run with the fast code and with the pre-processing script and let us know the time difference?

@trevorb1
Copy link
Member

@tniet Definitely! I was hoping to have the answer for you today on the time differences, but the pre-processed file has been running all day and hasn't solved yet. I will update on time differences once I have that information.

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