-
Notifications
You must be signed in to change notification settings - Fork 88
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restrictions on name attributes #918
Comments
That's a great question. I only have some random thoughts on this subject: I couldn't find anything written in the specification. Due to the generic nature of IFC and the fact that we're reusing IFC is an international standard so we have to consider that non-ascii use is probably valid. Backwards compatibility is highly considered, so even documentation changes constraining the use of attributes allowed in previous editions of the spec could be considered as prohibitive. In IDS it's possible to provide a regex patterns specifically for property set names if you were so inclined. There is a rule in the validation service (validate.buildingsmart.org) that checks that property set names with a Pset_ prefix conform to the list of predefined names. https://github.com/buildingSMART/ifc-gherkin-rules/blob/main/features/PSE001_Standard-properties-and-property-sets-validation.feature There is talks about extending that to other variations and permutations of pset in different cases and different separators. Other than that, I'd say to apply common sense when naming thing. |
just to confirm @aothms statement - the |
I totally recommend using emojis as much as possible. |
To give u some context:
Simply it is easier to Work with a project if the Data structure is set up in a manner that already considers this problems. Easiest definition would be the complete Avoidance of special character. But this has to be well defined. A List of Not useable letters or conventions would be nice. As Example: No Mathematical Operations or Symbols and punctuation
|
Maybe you could phrase that as a series of allowed unicode categories (https://www.compart.com/en/unicode/category) and a best practice rule could be formed out of that in the validation service (validate.buildingsmart.org). cc @evandroAlfieri |
@aothms thanks for the input, according to this list i would see "Lowercase Letter", "Uppercase Letter" and "__" as possible |
It is definitly an important topic. |
I know there is a restriction not to use the prefix "Pset_" for user defined property sets. Are there any other restrictions on names of property sets, properties etc?
The standard uses Pascal Case for PropertySets, using only the characters matching the regex
[a-zA-Z_
]. Is this to imply that user defined names MUST or SHOULD follow this style? Or, alternatively, is it permissible to use any technically allowed character sequence, except "Pset_" at the start of a property set name?The text was updated successfully, but these errors were encountered: