Skip to content

Expose shell_layer in animations #2092

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

Merged
merged 17 commits into from
Feb 19, 2025
Merged

Conversation

PProfizi
Copy link
Contributor

@PProfizi PProfizi commented Feb 17, 2025

Solves #1615
Solves #1435

This PR exposes the shell_layer selection logic through the animation APIs.
It also refactors imports in mapping_types as they were preventing me from updating signatures properly.

@PProfizi PProfizi added the bug Something isn't working label Feb 17, 2025
@PProfizi PProfizi requested a review from rafacanton February 17, 2025 16:59
@PProfizi PProfizi self-assigned this Feb 17, 2025
Copy link

codecov bot commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.61%. Comparing base (2bed7d2) to head (9b03dec).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2092      +/-   ##
==========================================
+ Coverage   86.39%   86.61%   +0.21%     
==========================================
  Files          90       90              
  Lines       10240    10250      +10     
==========================================
+ Hits         8847     8878      +31     
+ Misses       1393     1372      -21     

@PProfizi PProfizi changed the title Patch plots of some LS-Dyna fields Expose shell_layer in animations Feb 18, 2025
@PProfizi PProfizi force-pushed the fix/fields_container_animate_lsdyna branch from 5da4f0f to f16f050 Compare February 18, 2025 14:54
@PProfizi PProfizi linked an issue Feb 19, 2025 that may be closed by this pull request
3 tasks
@PProfizi PProfizi merged commit e2ed560 into master Feb 19, 2025
45 of 46 checks passed
@PProfizi PProfizi deleted the fix/fields_container_animate_lsdyna branch February 19, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DpfPlotter.add_field does not handle shell layers
2 participants