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

RelaxationStepLevelSetCorrectionInner doesn't work for 3d multi-resolution #663

Closed
WeiyiVirtonomy opened this issue Sep 20, 2024 · 5 comments · Fixed by #634
Closed

RelaxationStepLevelSetCorrectionInner doesn't work for 3d multi-resolution #663

WeiyiVirtonomy opened this issue Sep 20, 2024 · 5 comments · Fixed by #634

Comments

@WeiyiVirtonomy
Copy link
Collaborator

The relaxation using RelaxationStepLevelSetCorrectionInner produces weird particle distribution

Lattice:
Screenshot 2024-09-20 130113

After relaxation:
Screenshot 2024-09-20 130120

Gaps:
Screenshot 2024-09-20 130126

RelaxationStepInner doesn't have this problem:
image

There might be a bug in the 3d level-set shape of this beam

@Xiangyu-Hu
Copy link
Owner

@WeiyiVirtonomy yes. It seems like.
@Nafug22 could you have check on the implementation? Thanks.

@Xiangyu-Hu
Copy link
Owner

@WeiyiVirtonomy Could you make small test case for multi-resolution relaxation? Thanks,

@WeiyiVirtonomy
Copy link
Collaborator Author

@WeiyiVirtonomy Could you make small test case for multi-resolution relaxation? Thanks,

Sure. Would you like me to use this beam in the test, or would you prefer more complicated shapes?

We have a 3d teapot with surface refinement, and I wonder why that one works well. Could the problem lie in the shape generated by simtk?

@Xiangyu-Hu
Copy link
Owner

@WeiyiVirtonomy Just use this simple case.

@WeiyiVirtonomy
Copy link
Collaborator Author

The problem is now solved after fixing dW(h_ratio) of 3D and defining the level set refinement level correctly

@WeiyiVirtonomy WeiyiVirtonomy linked a pull request Dec 6, 2024 that will close this issue
3 tasks
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 a pull request may close this issue.

2 participants