buildingSMART / buildingSMART/IFC4.x-development

Inconsistent comma separated values in pset template definitions

Open
#1,071 8 comments 0 reactions 0 assignees View on GitHub
allocated-core ifc-update-out
Dominant language
Python
Stars
234
Forks
123
Avg merge
15h 4m
Merged PRs (30d)
5

Description

Some pset template definitions use comma separated values with spaces for their applicable entities, like so:

```
Pset_SpaceFireSafetyRequirements --> 'IfcSpace, IfcSpatialZone, IfcZone'
```

Others do not:

```
Pset_ConcreteElementGeneral --> 'IfcBeam,IfcBuildingElementProxy,IfcChimney,IfcColumn,IfcFooting,IfcMember,IfcPile,IfcPlate,IfcRailing,IfcRamp,IfcRampFlight,IfcRoof,IfcSlab,IfcStair,IfcStairFlight,IfcWall,IfcCivilElement'
```

The example given in the documentation does not use spaces.

I propose to remove all spaces in comma separated lists. Ping @berlotti @TLiebich @aothms

Contributor guide

Open the contributing guide

Research direction

The issue does not identify a file, test, or entry point; begin by locating the pset template definitions and the documentation example referenced in the report. Compare the applicable-entity lists across those definitions, then verify that the agreed comma-spacing convention is applied consistently and that the example remains aligned.

Written by the indexing model from the issue text.

Assessment

Domain
documentation
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.