Skip to content

Conversation

@josiahjohnston
Copy link
Contributor

  • Existing transmission lines now have their sunk costs carried into the objective function. Update example expectations accordingly. Issue Sunk costs #92
  • Ensure that executing switch solve --help outside of a project directory will not cause a crash.
  • Update outputs of the copperplate0 example (the only example that includes a full set of outputs).
  • Have setup.py use the version number from the switch_model package
  • Implement switch --version for getting the version number from the command-line tool.

@josiahjohnston
Copy link
Contributor Author

Good catch. I did some digging and changed that to a more robust method similar to what you suggest. Method 3 at https://packaging.python.org/single_source_version/

@josiahjohnston
Copy link
Contributor Author

josiahjohnston commented May 19, 2017

I added on some updates to min up and down time constraints where I tried to condens code and make it more readable.

Can I get a yay or nay on this stuff so I can close out this branch? Happy to move the up & downtime constraint fiddling to another pull request if that moves things along.

@mfripp
Copy link
Member

mfripp commented May 19, 2017

It looks mostly OK. However, I am concerned that you dropped the idea of having a band of capacity that does not participate in the minimum downtime constraint (previously calculated as committable_fraction). Without that term, the model can turn off some capacity, and then get around the min-downtime rule by turning on other capacity which is actually forced off by gen_max_commit_fraction, e.g., due to a maintenance outage.

…e objective function. Update example expectations accordingly.

Ensure that executing `switch solve --help` outside of a project directory will not cause a crash.
Update outputs of the copperplate0 example (the only example that includes a full set of outputs).
Have setup.py use the version number from the switch_model package
Implement `switch --version` for getting the version number from the command-line tool.
@josiahjohnston
Copy link
Contributor Author

Ok, I moved the min up/down time futzing to a separate branch and am going to close this one. I was concerned about leaving the transmission sunk costs bug up in the tagged version of the repo for so long. Matthias, can you either push this patched version to PyPi or give me access/show me the ropes so I can do it?

@josiahjohnston josiahjohnston merged commit b097bf1 into master May 19, 2017
@josiahjohnston josiahjohnston deleted the beta_bug_fixes branch May 19, 2017 18:33
staadecker added a commit that referenced this pull request Jan 28, 2023
Only include RPS module when needed
staadecker added a commit that referenced this pull request Jan 28, 2023
Only include RPS module when needed
staadecker added a commit that referenced this pull request Jan 29, 2023
Only include RPS module when needed
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.

3 participants