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

Element Quality: Skew #388

Closed
claireroche opened this issue Jun 28, 2024 · 2 comments · Fixed by #391
Closed

Element Quality: Skew #388

claireroche opened this issue Jun 28, 2024 · 2 comments · Fixed by #391
Assignees
Labels
new feature question Further information is requested

Comments

@claireroche
Copy link
Collaborator

  1. I think the skew quality over hexahedral element is not computed accurately.

In quality/src/HexQuality.cpp file:
https://github.com/LIHPC-Computational-Geometry/gmds/blob/main/quality/src/HexQuality.cpp
The skew() is not computed on normalized principal axes X1, X2, and X3 as it should according to Sandi'as report on Verdict geometric quality library (https://www.osti.gov/servlets/purl/901967).
I don't know if it is wanted or not. In the current state, the skewness of an element is not necessary between 0 and 1.

  1. It Could be useful to add skew computation on quads too, in the QuadQuality class, as described in Sandia's Verdict library.
@claireroche claireroche added question Further information is requested new feature labels Jun 28, 2024
@claireroche claireroche self-assigned this Jun 28, 2024
@claireroche
Copy link
Collaborator Author

Example to add in HexEvalTestSuite.h:
(https://github.com/LIHPC-Computational-Geometry/gmds/blob/main/quality/tst/HexEvalTestSuite.h)

HexQuality hq = HexQuality::build(math::Point(188,28,29), math::Point(191,28,29), math::Point(191,31,26), math::Point(188,31,26), math::Point(188,27,28), math::Point(191,27,28), math::Point(191,29,25), math::Point(188,29,25)); std::cout << "Skew: " << hq.skew() << std::endl;

The returned value is 12, while the hexahedron is shaped as illustrated below.

ROCHE_TIKZ.pdf

@nicolaslg
Copy link
Collaborator

Thanks for the report !

  • I concur that the implementation is faulty
  • we should add the skew for the quads

@claireroche claireroche linked a pull request Jul 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants