buildingSMART / buildingSMART/IFC4.x-development

Pset_Address.Pset_Address value for UserDefinedPurpose

Open
#964 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 an issue - Pset_Address.UserDefinedPurpose requires Purpose to be set to 'USERDEFINED' in case if user defined purpose is provided:

![Image](https://github.com/user-attachments/assets/28d7b493-9966-4c01-99d1-19f202e9e112)

But PEnum_AddressType doesn't have 'USERDEFINED' value:

![Image](https://github.com/user-attachments/assets/b17dbd2f-4f9b-4933-8754-dd0efbd48386)

I guess, it's just an overlook in PEnum_AddressType definition. But since psets were already released, maybe we can change description to assume 'OTHER' to be used instead of 'USERDEFINED'? Since IfcAddressTypeEnum doesn't have 'OTHER', I guess 'OTHER' have kind of the same meaning as 'USERDEFINED'.

![Image](https://github.com/user-attachments/assets/f9233a92-261d-461a-9dcf-906fbbd7c8ad)

I guess it would be inconsistent with how 'USERDEFINED' is used anywhere else, but I'm not sure if there's an alternative.

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.