search

NREL / BuildingMOTIF

Building Metadata OnTology Interoperability Framework (BuildingMOTIF). For models, see https://models.open223.info/
https://buildingmotif.readthedocs.io/
Other
50 stars 7 forks source link

Add test and fix for #326 #327

Closed gtfierro closed 2 months ago

gtfierro commented 5 months ago

Removes required params from the list of optional arguments. This occurs when the template inlining is performed and an optional template refers to a required parameter.

ACtually fixes #326 . The "237" was a mis-copy...

gtfierro commented 2 months ago

@TShapinsky everything is passing now (I haven't merged from develop yet because develop is broken). Can you approve when you get a chance? I added the additional test cases like you indicated. Thanks!