Upgrade miniconda - Githubissues

urbanopt / urbanopt-cli

URBANopt command line interface

https://docs.urbanopt.net/

Other

6 stars 2 forks source link

Upgrade miniconda #492

Closed vtnate closed 2 weeks ago

vtnate commented 2 weeks ago

Resolves #[issue number here]

Pull Request Description

Use a new version of miniconda. The previous version had an older version of pip included which couldn't handle new python packaging standards. This resolves that issue.
Use a new version of ThermalNetwork, to resolve the failing tests.

Since we set up miniconda, they have switched to date-version format, instead of semantic-version.

Checklist (Delete lines that don't apply)

[ ] Unit tests have been added or updated
[ ] All ci tests pass (green)
[ ] An issue has been created (which will be used for the changelog)
[x] This branch is up-to-date with develop

kflemin commented 2 weeks ago

@tanushree04, would you be able to test this on our windows machine? clearing out your python dependencies, running "bundle exec uo install_python" would be great!

tanushree04 commented 2 weeks ago

The command works!

I see some warnings while installing but i think they can be suppressed if needed.

Error installing: WARNING: The script pygmentize.exe is installed in 'C:\urbanopt-cli\example_files\python_deps\python-3.10\Scripts' which is not on PATH. Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location. WARNING: The script f2py.exe is installed in 'C:\urbanopt-cli\example_files\python_deps\python-3.10\Scripts' which is not on PATH. Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.

kflemin commented 2 weeks ago

agreed, I think those can be ignored. thanks for checking!!