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

Create finite element assembly for Springs elements #323

Open
wants to merge 27 commits into
base: main
Choose a base branch
from

Conversation

faisal-bhuiyan
Copy link
Collaborator

@faisal-bhuiyan faisal-bhuiyan commented Dec 19, 2024

Complements the work done in #321, i.e. completes the task description provided in springs element implementation. Specifically, finite element infrastructure was expanded to accommodate Springs elements creation and assembly in OpenTurbine.

Copy link
Collaborator

@ddement ddement 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 so far. You'll probably need to proliferate creating Springs and passing it to the Elements constructor before builds will go through

@faisal-bhuiyan faisal-bhuiyan self-assigned this Dec 22, 2024
@faisal-bhuiyan faisal-bhuiyan added the enhancement New feature or request label Dec 22, 2024
@faisal-bhuiyan faisal-bhuiyan marked this pull request as ready for review December 24, 2024 01:31
@faisal-bhuiyan
Copy link
Collaborator Author

@ddement I added unit tests for the residual and stiffness matrix assembly. While these look good, there is something wrong with the contribution to sparse matrix logic -- specifically this r-test is failing due to "KLU2 numeric factorization failed". Could you help with debugging this?

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

Successfully merging this pull request may close these issues.

2 participants