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

uo des_create issue #480

Open
glinn07 opened this issue Sep 17, 2024 · 3 comments · May be fixed by #486
Open

uo des_create issue #480

glinn07 opened this issue Sep 17, 2024 · 3 comments · May be fixed by #486
Labels
bug Something isn't working

Comments

@glinn07
Copy link

glinn07 commented Sep 17, 2024

Expected behavior

When I run uo des_create I expect the following line to work:

uo des_create -y systemparam.json -f example_project.json -n m3

Actual behavior

I receive the following output with an error at the end:

gl@hl-m2Working % uo des_create -y sampleUO/systemparam.json -f sampleUO/example_project.json -n sampleUO/m3 Top level ::CompositeIO is deprecated, require 'multipart/post' and useMultipart::Post::CompositeReadIOinstead! Top level ::Parts is deprecated, require 'multipart/post' and useMultipart::Post::Parts` instead!
Checking system.....
...python found at /Applications/URBANoptCLI_0.13.0/gems/ruby/2.7.0/gems/urbanopt-cli-0.13.0/example_files/python_deps/Miniconda-4.12.0/bin/python
...pip found at /Applications/URBANoptCLI_0.13.0/gems/ruby/2.7.0/gems/urbanopt-cli-0.13.0/example_files/python_deps/Miniconda-4.12.0/bin/pip
DEPENDENCIES RETRIEVED FROM FILE: [{:name=>"ThermalNetwork", :version=>"0.2.5"}, {:name=>"urbanopt-ditto-reader", :version=>"0.6.4"}, {:name=>"NREL-disco", :version=>"0.5.1"}, {:name=>"geojson-modelica-translator", :version=>"0.7.0"}]
...ThermalNetwork found with specified version 0.2.5
...urbanopt-ditto-reader found with specified version 0.6.4
...NREL-disco found with specified version 0.5.1
...geojson-modelica-translator found with specified version 0.7.0
Usage: uo_des create-model [OPTIONS] SYS_PARAM_FILE GEOJSON_FEATURE_FILE
[PROJECT_PATH]
Try 'uo_des create-model -h' for help.

Error: Got unexpected extra argument (Park/Working/sampleUO/m3)

I tried using the --des-name flag and got the same issue

If I try to run without specifying the des directory name I get the following:

gl@hl-m2 Working % uo des_create -y sampleUO/systemparam.json -f sampleUO/example_project.json
Top level ::CompositeIO is deprecated, require 'multipart/post' and use Multipart::Post::CompositeReadIO instead!
Top level ::Parts is deprecated, require 'multipart/post' and use Multipart::Post::Parts instead!
Error: option --des-name must be specified.

Additional context

Mac, running uo cli 0.13.0

@glinn07 glinn07 added the bug Something isn't working label Sep 17, 2024
@vtnate
Copy link
Collaborator

vtnate commented Sep 18, 2024

I think it is looking for a path, so instead of passing -n m3, try -n ./m3.

Yes, we currently require a model name name, so that --des-name must be specified is the expected behavior. However, I don't think we strictly need to require that; perhaps we'll relax that in the next version. It should be capable of providing a name by itself.

@glinn07
Copy link
Author

glinn07 commented Sep 20, 2024

Thanks Nate! Unfortunately didn't work:

gl@HL-M2 Working % uo des_create -y sampleUO/systemparam.json -f sampleUO/example_project.json -n ./m3
Top level ::CompositeIO is deprecated, require 'multipart/post' and use Multipart::Post::CompositeReadIO instead!
Top level ::Parts is deprecated, require 'multipart/post' and use Multipart::Post::Parts instead!
Checking system.....
...python found at /Applications/URBANoptCLI_0.13.0/gems/ruby/2.7.0/gems/urbanopt-cli-0.13.0/example_files/python_deps/Miniconda-4.12.0/bin/python
...pip found at /Applications/URBANoptCLI_0.13.0/gems/ruby/2.7.0/gems/urbanopt-cli-0.13.0/example_files/python_deps/Miniconda-4.12.0/bin/pip
DEPENDENCIES RETRIEVED FROM FILE: [{:name=>"ThermalNetwork", :version=>"0.2.5"}, {:name=>"urbanopt-ditto-reader", :version=>"0.6.4"}, {:name=>"NREL-disco", :version=>"0.5.1"}, {:name=>"geojson-modelica-translator", :version=>"0.7.0"}]
...ThermalNetwork found with specified version 0.2.5
...urbanopt-ditto-reader found with specified version 0.6.4
...NREL-disco found with specified version 0.5.1
...geojson-modelica-translator found with specified version 0.7.0
Usage: uo_des create-model [OPTIONS] SYS_PARAM_FILE GEOJSON_FEATURE_FILE
[PROJECT_PATH]
Try 'uo_des create-model -h' for help.

Error: Got unexpected extra argument (Park/Working/m3)

I also tried running directly within the project directory ("sampleUO") and uo totally crashes in that scenario

@glinn07
Copy link
Author

glinn07 commented Oct 4, 2024

Hi @vtnate I think I figured out the root of the issue

I am creating an example project but I am not specifying that it is a ghe project

I run the followings sequence of commands

uo create -p sampleUO

uo run -s sampleUO/baseline_scenario.csv -f sampleUO/example_project.json

uo des_params -y sampleUO/sys_params.json -s sampleUO/baseline_scenario.csv -f sampleUO/example_project.json -t 4G

These all work. In my mind I've created a plain vanilla set of example file and a hot water/cold water 4G system

But when I run uo des_create I get the following:

gl@HL-M2 Working % uo des_create -y sampleUO/sys_params.json -f sampleUO/example_project.json -n sampleUO/mm5
Top level ::CompositeIO is deprecated, require 'multipart/post' and use Multipart::Post::CompositeReadIO instead!
Top level ::Parts is deprecated, require 'multipart/post' and use Multipart::Post::Parts instead!
Checking system.....
...python found at /Applications/URBANoptCLI_0.13.0/gems/ruby/2.7.0/gems/urbanopt-cli-0.13.0/example_files/python_deps/Miniconda-4.12.0/bin/python
...pip found at /Applications/URBANoptCLI_0.13.0/gems/ruby/2.7.0/gems/urbanopt-cli-0.13.0/example_files/python_deps/Miniconda-4.12.0/bin/pip
DEPENDENCIES RETRIEVED FROM FILE: [{:name=>"ThermalNetwork", :version=>"0.2.5"}, {:name=>"urbanopt-ditto-reader", :version=>"0.6.4"}, {:name=>"NREL-disco", :version=>"0.5.1"}, {:name=>"geojson-modelica-translator", :version=>"0.7.0"}]
...ThermalNetwork found with specified version 0.2.5
...urbanopt-ditto-reader found with specified version 0.6.4
...NREL-disco found with specified version 0.5.1
...geojson-modelica-translator found with specified version 0.7.0
Usage: uo_des create-model [OPTIONS] SYS_PARAM_FILE GEOJSON_FEATURE_FILE
[PROJECT_PATH]
Try 'uo_des create-model -h' for help.

Error: Got unexpected extra argument (Park/Working/sampleUO/mm5)

I think maybe the issue is with 4G systems??? Because if I follow the path of the 5g_ghe system it works no problem...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants