Skip to content

Extruded plotting fixes #4553

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Extruded plotting fixes #4553

wants to merge 4 commits into from

Conversation

danshapero
Copy link
Contributor

Fixes for:

  • a bug in FunctionPlotter that was giving incorrect plot results for functions on extruded meshes
  • default shading when using tripcolor with DG0 does the wrong thing when you try to make a movie
  • unclear error when you try to do triplot on an extruded mesh

@danshapero
Test plotting on extruded mesh with multiple layers
2516652
@danshapero
Fix plotting for extruded meshes
6da4fe0 
.cell_node_list doesn't give the complete result for extruded meshes
with multiple layers, it relies on the fact that you can increment it to
get the node list for the remaining layers; the result is that you still
end up plotting the field, but only the lowest layer. Fixes the
accounting for multiple layers.
@danshapero
Switch default shading with tripcolor
4b764cb 
Using flat shading with DG0 functions messes up the indexing when you
try to make a movie; we actually want gouraud all the time.
@danshapero
Add error for triplot extruded
4e6c238
@danshapero danshapero force-pushed the dan/extruded-plotting-fixes branch from 99c96a8 to 4e6c238 Compare September 9, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@danshapero