Skip to content

Add GSA10 exception to UnitType - #353

Open
johannaisak wants to merge 3 commits into
developfrom
GSA_Toolkit-#352-Unit-conversion-in-GSA10
Open

Add GSA10 exception to UnitType#353
johannaisak wants to merge 3 commits into
developfrom
GSA_Toolkit-#352-Unit-conversion-in-GSA10

Conversation

@johannaisak

@johannaisak johannaisak commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

Assigns correct unit types for GSA10.2

NOTE: Depends on

Issues addressed by this PR

Closes #352

Test files

https://burohappold.sharepoint.com/:f:/r/sites/BHoM/02_Current/12_Scripts/02_Pull%20Request/BHoM/GSA_Toolkit/%23353-Unit-conversion-in-GSA10?csf=1&web=1&e=QO6JQA

Changelog

Additional comments

Assigns correct unit types for GSA10
@johannaisak johannaisak added status:do-not-merge For instance, test PR, for discussion, or dependant PRs not ready for merge type:bug Error or unexpected behaviour labels Apr 10, 2026
@johannaisak johannaisak self-assigned this Apr 10, 2026
@johannaisak

Copy link
Copy Markdown
Contributor Author

@BHoMBot check installer

@bhombot-ci

bhombot-ci Bot commented Apr 10, 2026

Copy link
Copy Markdown

@johannaisak to confirm, the following actions are now queued:

  • check installer

@johannaisak

Copy link
Copy Markdown
Contributor Author

@BHoMBot check installer

@bhombot-ci

bhombot-ci Bot commented Apr 10, 2026

Copy link
Copy Markdown

@johannaisak to confirm, the following actions are now queued:

  • check installer

@johannaisak

Copy link
Copy Markdown
Contributor Author

@BHoMBot check installer

@johannaisak
johannaisak requested review from RubenKoopman and removed request for MaxwellGarnett July 27, 2026 09:57

@RubenKoopman RubenKoopman left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The GSA output for the bar force, displacement, stress and strain has been compared to the results pulled in Grasshopper. The units are correctly pulled.

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

Labels

status:do-not-merge For instance, test PR, for discussion, or dependant PRs not ready for merge type:bug Error or unexpected behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unit conversion in GSA10

2 participants