search

NREL / OpenStudio

OpenStudio is a cross-platform collection of software tools to support whole building energy modeling using EnergyPlus and advanced daylight analysis using Radiance.
https://www.openstudio.net/
Other
484 stars 185 forks source link

Failed to Infer Measure Name #5215

Open kflemin opened 1 month ago

kflemin commented 1 month ago

When running the openstudio CLI measure -u call on a measure, I often get this error: "Failed to infer measure name from '/path/to/measure/measure.rb'". I get it on Python measures as well as Ruby measures. This error also ends up in the xml file in an tag. For example: https://github.com/NREL/OpenStudio-workflow-gem/blob/39c84e45f152446a92379a501807e2691dba15ab/spec/files/script_error_osw/measures/IncreaseWallRValue/measure.xml#L4

Measures contains such errors do not get uploaded to the BCL. I don't think they work in the OS apps either.

Ruby code that triggers the error: https://github.com/NREL/OpenStudio/blob/ba1b94b6542c1f1d4cdcdb775886852b9a82bc4f/ruby/engine/RubyEngine.cpp#L154-L203 Python code that triggers the error: https://github.com/NREL/OpenStudio/blob/eb043d23f746e8df3b2819ac43eeb19958973e0c/python/engine/PythonEngine.cpp#L286-L310

I'd like to understand better why this error is being triggered...I can't tell exactly from the code. It would seem like there's a typo or not-recommended pattern in the class name, but that doesn't actually seem to be the case.

Another piece of information: I can sometimes update a measure with openstudio measure -u which triggers the error (in addition to doing other cleanup with the files and checksums but doesn't touch name/class_name), remove the error from the measure.xml, re-update the measure with openstudio measure -u again, and now it works?!

thanks!

macumber commented 3 weeks ago

I think this is related to https://github.com/NREL/OpenStudio/issues/5212