We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
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.
skew()
QuadQuality
The text was updated successfully, but these errors were encountered:
Example to add in HexEvalTestSuite.h: (https://github.com/LIHPC-Computational-Geometry/gmds/blob/main/quality/tst/HexEvalTestSuite.h)
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
Sorry, something went wrong.
Thanks for the report !
claireroche
Successfully merging a pull request may close this issue.
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.
QuadQuality
class, as described in Sandia's Verdict library.The text was updated successfully, but these errors were encountered: