buildingSMART / buildingSMART/IFC4.x-development

PerformanceHistory in IfcPropertySetTemplate.ApplicableEntity

Open
#935 2 comments 0 reactions 0 assignees View on GitHub
after-NWI iso-Te
Dominant language
Python
Stars
234
Forks
123
Avg merge
15h 4m
Merged PRs (30d)
5

Description

Noticed that generating pset templates for IFC4X3 I've missed "PerformanceHistory" in ApplicableEntity.

E.g. Pset_CoilPHistory is using "IfcCoil" instead of "IfcCoil[PerformanceHistory]".

https://github.com/buildingSMART/IFC4.3.x-output/blob/1ec4dec449fda98ae8e1409ac726348d9985ab05/Pset_IFC4X3.ifc#L178-L180

From the docs (https://ifc43-docs.standards.buildingsmart.org/IFC/RELEASE/IFC4x3/HTML/lexical/IfcPropertySetTemplate.htm) it seems that it should indeed be expanded "[PerformanceHistory]" for this case:

![Image](https://github.com/user-attachments/assets/1e16479c-d2c2-4780-96a1-387569beba94)

But then I've investigated official pset templates for IFC4_ADD2 (https://standards.buildingsmart.org/IFC/RELEASE/IFC4/ADD2_TC1/HTML/annex/annex-a/general-usage/IFC4_ADD2.ifc)
and they're using "IfcCoil" without "[PerformanceHistory]" too. Don't know if it's intended or just an error.

![Image](https://github.com/user-attachments/assets/69bbf61d-8fe1-459b-93b1-0bb490893fa6)

And it raised a couple questions for me:
1) In what cases ApplicableEntity really should have "[PerformanceHistory]" addition?
2) Is there really any use for "[PerformanceHistory]" addition if there is "PSET_PERFORMANCEDRIVEN" in IfcPropertySetTemplate.TemplateType which seems to imply the same?
![Image](https://github.com/user-attachments/assets/a1d418cd-83e0-4e0c-b0f7-831548293881)

But I guess it still can be useful for other template types:
- PSET_OCCURRENCEDRIVEN and ApplicableEntity "IfcWindow,IfcWindow[PerformanceHistory]" pset will be assignable to both IfcWindow and IfcPerformanceHistory->IfcWindow but there's no such examples of these kind of psets in the schema.
- QTO_OCCURRENCEDRIVEN - since there is no QTO_PERFORMANCEDRIVEN, so ApplicableEntity can help limiting some quantity set to be used only with IfcPerformanceHistory

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.