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

Update assembly.py #309

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

Conversation

thisisapple
Copy link

When the user adds a node, the original element will be deleted. The SystemElements object cannot pass validation because a non-existing element was referenced. A key error will occur. To avoid this error, I use the enumerate() method.
This can avoid the the key error. The original code will reference the deleted key in SystemElements.element_map.

When the user adds a node, the original element will be deleted. The SystemElements object cannot pass validation because a non-existing element was referenced. An error will occur.
To avoid this error, I use the enumerate() method.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you so much for contributing to the open-source anaStruct project! Your contribution will help thousands of engineers work more efficiently and accuractely.

Now that you've created your first pull request here, please don't go away; take a look at the bottom of this page for the automated checks that should already be running. If they pass, great! If not, please click on 'Details' and see if you can fix the problem they've identified. A maintainer should be along shortly to review your pull request and help get it added to anaStruct!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant