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

Extract Mesh VonMises Stresses #471

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

GCRA101
Copy link
Contributor

@GCRA101 GCRA101 commented Sep 10, 2024

No Von Mises Flexural Moment gets extracted since it's not a direct output given by ETABS.
It could be possibly computed within the BHoM in case it's required.
The Von Mises Moment is not returned by ETABS natively and it must be computed directly in the BHoM from the Von Mises stresses diagram.
Thickness of panels is extracted differently depending on the type of deck/slab/wall section property assigned to the panel.
The ETABS API provides different functions for each of their types and they must be all called somehow until the correct one gets hit.
The control parameter is the integer ouput ret of the ETABS API functions (0=Worked Fine, 1=Failed)
@GCRA101 GCRA101 added type:feature New capability or enhancement status:do-not-merge For instance, test PR, for discussion, or dependant PRs not ready for merge labels Sep 10, 2024
@GCRA101 GCRA101 added this to the BHoM 7.4 β MVP milestone Sep 10, 2024
@GCRA101 GCRA101 self-assigned this Sep 10, 2024
@GCRA101
Copy link
Contributor Author

GCRA101 commented Sep 10, 2024

@IsakNaslundBh, @Chrisshort92 , @peterjamesnugent, just to be clear, this Pull Request is for the next BHoM 7.4 Milestone in autumn. Not the one coming up now in september. ;)

@peterjamesnugent peterjamesnugent removed the status:do-not-merge For instance, test PR, for discussion, or dependant PRs not ready for merge label Oct 1, 2024
Copy link

@Chrisshort92 Chrisshort92 left a comment

Choose a reason for hiding this comment

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

Functionality Tested with provided test files and working as expected. approved for merge

@Chrisshort92
Copy link

Chrisshort92 commented Oct 14, 2024

@BHoMBot check required

Copy link

bhombot-ci bot commented Oct 14, 2024

@Chrisshort92 sorry, I didn't understand.
Was that comment an instruction for me? If so, could you state again what check you would like me to do?
For a list of available instructions, please see this wiki page.

Copy link

bhombot-ci bot commented Oct 14, 2024

@Chrisshort92 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 64 requests in the queue ahead of you.

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 to Extract Mesh VonMises Stresses
3 participants