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

Negative Jacobians encountered in dynamic cuboid problem #51

Open
ikalash opened this issue Feb 15, 2025 · 4 comments
Open

Negative Jacobians encountered in dynamic cuboid problem #51

ikalash opened this issue Feb 15, 2025 · 4 comments
Assignees

Comments

@ikalash
Copy link
Collaborator

ikalash commented Feb 15, 2025

When running the single-domain AHeaD dynamic cuboid problem, negative Jacobians are encountered when adaptive time-stepping is disabled: https://github.com/sandialabs/Norma.jl/blob/main/examples/ahead/single/cuboid/dynamic/cuboid.yaml . This happens with dt = 0.01 and larger dt, even though the larger dts should work. When running the problem with adaptive time-stepping, the velocity and acceleration fields look wrong, as shown in the images below.

It is not 100% clear if the problem had issues from the beginning or if this started recently. It definitely ran initially with dt = 0.01 and no adaptive time-stepping.

Image

Image

@lxmota
Copy link
Collaborator

lxmota commented Feb 18, 2025

I think this issue and #50 are related, and that the problem is older, because even reverting to 060269f shows incorrect velocities and accelerations for me. I'm not sure how the tests pass. Working on this.

@ikalash
Copy link
Collaborator Author

ikalash commented Feb 18, 2025

@lxmota they could be related. I have been focusing on the clamped problem issue. I've narrowed it down a bit more where this commit is bad 25f3b46 whereas this commit is good c72ab22 .

@ikalash
Copy link
Collaborator Author

ikalash commented Feb 18, 2025

Wait, 060269f is a merge of Brian's work on contact + inclined support. The clamped problem was still OK after that merge.

@ikalash
Copy link
Collaborator Author

ikalash commented Feb 18, 2025

Also, I think we should add a test for the clamped problem once this is fixed where the disp, velo and acce are all compared to a reference. This will catch issues like this coming up in the future, as the clamped problem is a pretty stringent test case.

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

No branches or pull requests

2 participants