Skip to content

Conversation

@dipinknair
Copy link
Collaborator

@dipinknair dipinknair commented Oct 16, 2023

  • Example requires (Mechanical by default) at least 4 core where GitHub runner had only 2.
  • Adding simple cantilever structural optimization to demonstrate pymechanical capabilities.

image

Topology optimized result
image

@github-actions github-actions bot added the enhancement New features or code improvements label Oct 16, 2023
@dipinknair dipinknair marked this pull request as ready for review October 16, 2023 15:05
@dipinknair dipinknair requested a review from klmcadams October 16, 2023 15:59
@dipinknair dipinknair self-assigned this Oct 16, 2023
@klmcadams klmcadams requested a review from PipKat October 16, 2023 17:19
dipinknair and others added 4 commits October 16, 2023 13:09
@github-actions github-actions bot added the maintenance Package and maintenance related label Oct 16, 2023
@dipinknair dipinknair requested a review from klmcadams October 17, 2023 01:52
Copy link
Collaborator

@vnamdeo vnamdeo left a comment

Choose a reason for hiding this comment

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

Overall looks good.

@vnamdeo
Copy link
Collaborator

vnamdeo commented Oct 17, 2023

  • Example requires (Mechanical by default) at least 4 core where GitHub runner had only 2.
  • Adding simple cantilever structural optimization to demonstrate pymechanical capabilities.

image image

@dipinknair : Could you please check the topo results as it looks more symmetric in terms of mass removal. Near the support condition, not expecting much mass removal. Anyway, will discuss more on this. Thanks for adding this example.

@dipinknair dipinknair requested a review from vnamdeo October 17, 2023 14:42
Copy link
Member

@PipKat PipKat left a comment

Choose a reason for hiding this comment

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

@klmcadams My editorial review consisted mostly of changing inital capital letters to lowercase letters.

Co-authored-by: Kathy Pippert <84872299+PipKat@users.noreply.github.com>
@dipinknair dipinknair requested a review from PipKat October 17, 2023 15:15
Copy link
Collaborator

@vnamdeo vnamdeo left a comment

Choose a reason for hiding this comment

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

Looks good.

@dipinknair dipinknair merged commit 9c6219f into main Oct 17, 2023
@dipinknair dipinknair deleted the feat/topo-example branch October 17, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or code improvements maintenance Package and maintenance related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants