Skip to content

Add UnstructuredQuadMesh to load_mesh_serial#575

Merged
sloede merged 5 commits intomainfrom
fix_unstructured_plotting
May 13, 2021
Merged

Add UnstructuredQuadMesh to load_mesh_serial#575
sloede merged 5 commits intomainfrom
fix_unstructured_plotting

Conversation

@andrewwinters5000
Copy link
Copy Markdown
Member

Necessary structure to reconstruct the unstructured quadrilateral mesh for conversion to vtk via Trixi2Vtk

@andrewwinters5000 andrewwinters5000 requested review from ranocha and sloede and removed request for ranocha May 13, 2021 05:51
@codecov
Copy link
Copy Markdown

codecov bot commented May 13, 2021

Codecov Report

Merging #575 (abc2b29) into main (ca0a78f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head abc2b29 differs from pull request most recent head 611d6e6. Consider uploading reports for the commit 611d6e6 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #575   +/-   ##
=======================================
  Coverage   93.72%   93.72%           
=======================================
  Files         138      138           
  Lines       13837    13840    +3     
=======================================
+ Hits        12969    12972    +3     
  Misses        868      868           
Flag Coverage Δ
unittests 93.72% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/callbacks_step/save_restart_dg.jl 88.29% <100.00%> (-0.13%) ⬇️
src/mesh/mesh_io.jl 95.75% <100.00%> (+0.10%) ⬆️

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 222fcd8...611d6e6. Read the comment docs.

@sloede sloede merged commit c1e6beb into main May 13, 2021
@sloede sloede deleted the fix_unstructured_plotting branch May 13, 2021 07:39
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