Skip to content

adding subtypes of door and transportationmechanism#650

Open
JValiunas wants to merge 2 commits intomasterfrom
archphys_1.0.4
Open

adding subtypes of door and transportationmechanism#650
JValiunas wants to merge 2 commits intomasterfrom
archphys_1.0.4

Conversation

@JValiunas
Copy link
Copy Markdown
Collaborator

draft PR for some missing ArchPhys classes

@JValiunas JValiunas requested a review from a team as a code owner January 27, 2026 11:19
</Target>
</ECRelationshipClass>

<ECEntityClass typeName="Railing" displayLabel="Stair Flight" modifier="None" description="a frame assembly adjacent to circulation spaces where it is used in lieu of, or to compliment walls. An optional physical support to prevent injury or damage, either by falling or collision.">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: Fix displayLabel.

<Class class="RailingType"/>
</Target>
</ECRelationshipClass>

Copy link
Copy Markdown
Collaborator Author

@JValiunas JValiunas Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need a Landing class - could be a metallic frame or timber

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

discussion led us to believe "Landing" would be spatial in nature, not part of this schema.

We need a way to associate slab with stair. consult with structural domain authors

<BaseClass>TransportationMechanism</BaseClass>
</ECEntityClass>

<ECEntityClass typeName="Elevator" displayLabel="Elevator" modifier="None" description="Elevator or lift to move people or goods vertically.">
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@diegoalexdiaz look into elevators

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants