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

Implement export of projected fields using the projector #390

Merged
merged 1 commit into from
Jan 11, 2022

Conversation

fredrikekre
Copy link
Member

Implement export of projected fields using the projector and simplify vtk_point_data with tensor output.

and simplify vtk_point_data with tensor output.
@codecov-commenter
Copy link

codecov-commenter commented Oct 12, 2021

Codecov Report

Merging #390 (e04b22a) into master (636f273) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #390      +/-   ##
==========================================
+ Coverage   89.78%   89.82%   +0.04%     
==========================================
  Files          22       22              
  Lines        2701     2713      +12     
==========================================
+ Hits         2425     2437      +12     
  Misses        276      276              
Impacted Files Coverage Δ
src/Dofs/DofHandler.jl 86.10% <ø> (ø)
src/Dofs/MixedDofHandler.jl 86.97% <100.00%> (ø)
src/Export/VTK.jl 91.07% <100.00%> (-1.36%) ⬇️
src/L2_projection.jl 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 636f273...e04b22a. Read the comment docs.

@kimauth
Copy link
Member

kimauth commented Jan 6, 2022

Changing the export order of regular tensors is sort of breaking, is it? In Paraview the entries will be "shuffled" up compared to the previous export and it isn't obvious which one is which from what you can see in Paraview.

Other than that, this looks good.

@fredrikekre
Copy link
Member Author

I made a PR to label non-symmetric tensors too in ParaView: https://gitlab.kitware.com/paraview/paraview/-/merge_requests/5442

@fredrikekre fredrikekre merged commit 5994169 into master Jan 11, 2022
@fredrikekre fredrikekre deleted the fe/export branch January 11, 2022 16:45
koehlerson pushed a commit that referenced this pull request Apr 22, 2022
and simplify vtk_point_data with tensor output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants