Skip to content

Conversation

@josiahjohnston
Copy link
Contributor

…default is to print out all decision variables).

…default is to print out all decision variables).
@josiahjohnston josiahjohnston requested a review from mfripp October 4, 2019 18:04
@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