search

mattdoiron / idfplus

IDF+ is an enhanced, cross-platform editor for EnergyPlus input files.
GNU General Public License v3.0
12 stars 1 forks source link

Class ZoneHVACEquipmentConnections includes other class objects #39

Closed mattdoiron closed 4 years ago

mattdoiron commented 4 years ago

Class ZoneHVACEquipmentConnections inclues the following classes which should be separate:

mattdoiron commented 4 years ago

Hi, the ZoneHVAC:EquipmentConnections object connects to either a node or a ZoneHVAC:EquipmentList. Do you mean that you are seeing fans and coils listed in a dropdown as well?

mattdoiron commented 4 years ago

No, its just a GUI issue as far as I've seen. The objects in the classes above are all grouped under the ZoneHVACEquipmentConnections class in the class list instead of being separated like in the IDF Editor.

IDFPlus_Issue33-1.png

IDFPlus_Issue33.png

mattdoiron commented 4 years ago

Ahh! I'm seeing the same issue, but didn't notice it before. I think the problem is that the group tag in the idd file is sometimes capitalized and sometimes not. I'll change it to check for both versions of the capitalization. Thanks for reporting this!

mattdoiron commented 4 years ago

The IDD file is actually quite inconsistent with regards to capitalization and lots of tags are affected, not just the group tag. The parser needs to be adjusted to allow for more loose capitalization rules. This is somewhat related to Issue #26, but for the IDD instead. Also related to Issue #32.

mattdoiron commented 4 years ago

Parser was applying very strict rules to capitalization of IDD tags. This meant is was missing important tags such as the group tag. This is fixes #33 but user will need to manually delete the already-processed IDD file so this is also related to Issue #32.

→ <<cset 30874180342b>>