Skip to content
This repository was archived by the owner on Aug 18, 2021. It is now read-only.

Add 3d functionality to Trixi2Img #18

Merged
merged 23 commits into from
Oct 20, 2020
Merged

Conversation

efaulhaber
Copy link
Member

@efaulhaber efaulhaber commented Oct 7, 2020

@codecov
Copy link

codecov bot commented Oct 8, 2020

Codecov Report

Merging #18 into master will increase coverage by 6.35%.
The diff coverage is 95.38%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   60.23%   66.59%   +6.35%     
==========================================
  Files           5        5              
  Lines         420      476      +56     
==========================================
+ Hits          253      317      +64     
+ Misses        167      159       -8     
Impacted Files Coverage Δ
src/Trixi2Img.jl 100.00% <ø> (ø)
src/io.jl 94.00% <83.33%> (-1.56%) ⬇️
src/interpolate.jl 88.63% <97.95%> (+5.50%) ⬆️
src/convert.jl 85.13% <100.00%> (+0.62%) ⬆️
src/interpolation.jl 40.63% <0.00%> (+4.27%) ⬆️

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 beee53c...112464b. Read the comment docs.

@efaulhaber efaulhaber marked this pull request as ready for review October 9, 2020 11:32
@efaulhaber efaulhaber requested a review from sloede October 9, 2020 11:45
A lot of variable-independent code was executed again for every 
variable.
Copy link
Member

@sloede sloede left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work! I left a few comments; please re-request a review once you're done or ping me if you have a question in the meantime.

@efaulhaber efaulhaber requested a review from sloede October 14, 2020 13:40
Copy link
Member

@sloede sloede left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I especially liked the thorough testing 👍 Once the one minor comment is addressed, and you re-request a review from me, I will merge this.

@efaulhaber efaulhaber requested a review from sloede October 20, 2020 09:18
@sloede sloede merged commit f60dc83 into trixi-framework:master Oct 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable Trixi2Img to plot results from 3D simulations
3 participants