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
Fix active normals for properties point_normals
, cell_normals
, and method plot_normals
with smooth_shading
#6062
Conversation
Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6062 +/- ##
==========================================
+ Coverage 96.98% 96.99% +0.01%
==========================================
Files 141 141
Lines 24639 24640 +1
==========================================
+ Hits 23895 23899 +4
+ Misses 744 741 -3 |
Note: the So, that test was failing because with this PR, the "stock" normals are now used, resulting in a different rendering. The new rendered image is updated in 2bfc4a1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. This is largely a bug, but it also includes an element of breaking change, so it will be labeled and made known in the release notes.
…d method `plot_normals` with `smooth_shading` (#6062) * Fix point_normals and cell_normals properties * Add plot_normals test * Add fix for smooth shading * Update test * Update docs * Apply suggestions from code review Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com> * Update backface param test image --------- Co-authored-by: Tetsuo Koyama <tkoyama010@gmail.com>
Resolves #6061
Resolves #2921
Resolves #6058