urbanopt / urbanopt-cli

URBANopt command line interface
https://docs.urbanopt.net/
Other
6 stars 2 forks source link

Install exact python dependency versions, and bump ThermalNetwork version #457

Closed vtnate closed 8 months ago

vtnate commented 8 months ago

Resolves #[issue number here]

Pull Request Description

ThermalNetwork version has been updated. We grab the newest patch version when we install, because we use tilde-version here. However, this if statement wants the version to match exactly, including patch version, so we output the "incorrect version" message. In this case, that's an erroneous message, but perhaps not always? During heavy development, I can imagine that sometimes the patch level could contain breaking changes. Or should we change the logic for that incorrect version message?

Decided to install exact dependency versions to eliminate this problem. In the distant future when versions are more stable we can revisit this.

Checklist (Delete lines that don't apply)