Skip to content

Allow Pull of FE Mesh Objects - #353

Open
GCRA101 wants to merge 11 commits into
developfrom
SAP2000_Toolkit-#352-AllowPullOfMeshObjects
Open

Allow Pull of FE Mesh Objects#353
GCRA101 wants to merge 11 commits into
developfrom
SAP2000_Toolkit-#352-AllowPullOfMeshObjects

Conversation

@GCRA101

@GCRA101 GCRA101 commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

@GCRA101 GCRA101 self-assigned this Jul 29, 2026
@GCRA101 GCRA101 added the type:feature New capability or enhancement label Jul 29, 2026

@peterjamesnugent peterjamesnugent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A few changes making use of existing methods.

Comment thread SAP2000_Adapter/Convert/ToBHoM/Panel.cs Outdated
Comment thread SAP2000_Adapter/Convert/ToBHoM/Panel.cs Outdated
Comment thread SAP2000_Adapter/SAP2000_Adapter.csproj
Comment thread SAP2000_Adapter/CRUD/Read/Mesh.cs Outdated
Comment thread SAP2000_Adapter/CRUD/Read/Mesh.cs Outdated
Comment thread SAP2000_Adapter/CRUD/Read/Mesh.cs Outdated
Comment thread SAP2000_Adapter/CRUD/Read/Mesh.cs
@GCRA101
GCRA101 requested a review from peterjamesnugent July 30, 2026 13:32
@GCRA101

GCRA101 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author

@BHoMBot check required

@bhombot-ci

bhombot-ci Bot commented Jul 30, 2026

Copy link
Copy Markdown

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

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check core
  • check null-handling
  • check serialisation
  • check versioning
  • check installer

There are 404 requests in the queue ahead of you.

@peterjamesnugent peterjamesnugent left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Everything else looks good, just one change to use Localisation_Toolkit for unit conversion.

Comment thread SAP2000_Adapter/Convert/ToBHoM/Panel.cs Outdated
LukeBastianBH
LukeBastianBH previously approved these changes Jul 31, 2026

@LukeBastianBH LukeBastianBH 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.

Successfully pulled panels and meshes using the provided GH script in Rhino 6 and the associated installer. Also pulled bars as independent check.

@GCRA101

GCRA101 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

Successfully pulled panels and meshes using the provided GH script in Rhino 6 and the associated installer. Also pulled bars as independent check.

fyi, @peterjamesnugent

@GCRA101
GCRA101 requested a review from peterjamesnugent July 31, 2026 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New capability or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow Pull of Mesh Objects

3 participants