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

Add option to skip saving each decision variable in a separate file (… #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

josiahjohnston
Copy link
Contributor

…default is to print out all decision variables).

…default is to print out all decision variables).
@josiahjohnston
Copy link
Contributor Author

Partially addresses issue #104

@mfripp
Copy link
Member

mfripp commented Nov 6, 2019

I like this, but I think the terminology may confuse people. We generally use "--no-something" instead of "--skip-something" in our flags. But also people probably won't know what's meant by "generic outputs".

I would recommend making this parallel with --save-expression[s] (which defaults to none), by creating a --save-variables flag which defaults to all. Then people could turn off all variables with --save variables none and then turn interesting ones back on with --save-variable BuildGen or --save-variables BuildGen BuildStorageEnergy.

A couple of longer-term improvements (don't have to be done now):

  • merge --save-expression and --save-variable (unclear to users why one is saved and not the other)
  • create groups other than all and none that people can specify (e.g., all per-period vars/exprs, all per-timepoint ones, all one-off values, etc.)

staadecker pushed a commit that referenced this pull request Jan 28, 2023
Script to fix derating factor for transmission lines
staadecker pushed a commit that referenced this pull request Jan 28, 2023
Script to fix derating factor for transmission lines
staadecker pushed a commit that referenced this pull request Jan 29, 2023
Script to fix derating factor for transmission lines
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 this pull request may close these issues.

2 participants