Skip to content
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

CNX-573 CSIOpening #3632

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

CNX-573 CSIOpening #3632

wants to merge 1 commit into from

Conversation

bjoernsteinhagen
Copy link
Contributor

@bjoernsteinhagen bjoernsteinhagen commented Sep 24, 2024

Ticket

Please refer here for more information: CNX-573

Description & Motivation

In creating content for Revit & Grasshopper to ETABS, I noticed that the CSIOpening component is essentially useless. We write and append CSIOpening information to the AreaObj, but this information is not used when receiving in ETABS.

Definition in Grasshopper:

image

Receiving in ETABS:
These AreaObjs should have Yes for attribute Opening

image

Changes:

Looking at the API docs:

image

We need to SetOpening in the same way we assign DiaphragmAssignment / CSIAreaSpring:

image

For AreaToSpeckle we target CSIOpening and not property (otherwise ETABS ➡️ Speckle ➡️ ETABS won't be compatible).

image

Validation of changes:

Grasshopper ➡️ Speckle ➡️ ETABS

image

ETABS ➡️ Speckle ➡️ ETABS

image

Revit ➡️ Speckle ➡️ ETABS

image

ETABS ➡️ Speckle ➡️ Revit

⚠️ NOTE: this didn't work previously, and is still not working.

Previously:

image

Now:

image

Checklist:

  • My pull request follows the guidelines in the Contributing guide?
  • My pull request does not duplicate any other open Pull Requests for the same update/change?
  • My commits are related to the pull request and do not amend unrelated code or documentation.
  • My code follows a similar style to existing code.
  • I have added appropriate tests.
  • I have updated or added relevant documentation.

Comments

This is messy. We need to revisit at a later stage.

image

CSIOpening attribute is created when going from AreaToSpeckle, but not used when going from AreaToNative.
@bjoernsteinhagen bjoernsteinhagen added bug Something isn't working grasshopper issues related to the grasshopper connector. CSI labels Sep 24, 2024
@bjoernsteinhagen bjoernsteinhagen self-assigned this Sep 24, 2024
Copy link

linear bot commented Sep 24, 2024

CNX-573 CSIOpening

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CSI grasshopper issues related to the grasshopper connector.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant