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

Output script generating a plot with solve time taken for each time step #196

Merged
merged 5 commits into from
Jun 25, 2020

Conversation

abhimishr
Copy link
Member

Description

Added a starting script which can generate a plot detailing how many minutes each time step took during the model run.

Change in model behavior: None

Functionality:

Such plot can help identify if certain time steps take too long to solve especially with elastic food demand turned on.

Sample:

Solve_timing

Performance loss: None
This is an output script and takes a few seconds to run. Can also be added to cfg$output vector.

@abhimishr abhimishr changed the title Develop Output script generating a plot with solve time taken for each time step Jun 24, 2020
@abhimishr abhimishr added enhancement New feature or request Low risk Low risk Minor Smaller modifications Suggestion labels Jun 24, 2020
Copy link
Member

@tscheypidi tscheypidi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest to put it into the "extra" selection and not to define the position (just remove that line)

@abhimishr
Copy link
Member Author

I would suggest to put it into the "extra" selection and not to define the position (just remove that line)

Done via 27ee4b8

@tscheypidi tscheypidi merged commit ae5b7ee into magpiemodel:develop Jun 25, 2020
abhimishr added a commit to abhimishr/magpie that referenced this pull request Jun 25, 2020
Merge pull request magpiemodel#196 from abhimishr/develop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Low risk Low risk Minor Smaller modifications Suggestion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants