Skip to content

Add tutorial on visualization with Plots.jl and interpolation - #1298

Open
efaulhaber wants to merge 9 commits into
mainfrom
ef/tut-visualization-plots
Open

Add tutorial on visualization with Plots.jl and interpolation#1298
efaulhaber wants to merge 9 commits into
mainfrom
ef/tut-visualization-plots

Conversation

@efaulhaber

@efaulhaber efaulhaber commented Aug 20, 2026

Copy link
Copy Markdown
Member

@efaulhaber efaulhaber left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Codex Review

The PR adds a Plots-based visualization tutorial, integrates it into the generated documentation, and links it from the tutorial and visualization pages. The tutorial's VTK export can fail in a clean working directory, and its particle plot applies fluid-only values to every plotted system.

This is an AI-generated code review. Please verify the findings and summary before acting on them.
Review generated by codex-pr-review

Comment thread docs/literate/src/tut_visualization.jl
Comment thread docs/literate/src/tut_visualization.jl Outdated
Comment thread docs/literate/src/tut_visualization.jl
Comment thread docs/literate/src/tut_visualization.jl
Comment thread docs/literate/src/tut_visualization.jl
Comment thread docs/literate/src/tut_visualization.jl
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 27 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.44%. Comparing base (26c5fcc) to head (df6c37e).

Files with missing lines Patch % Lines
src/visualization/recipes_plots.jl 0.00% 27 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1298      +/-   ##
==========================================
- Coverage   90.49%   90.44%   -0.05%     
==========================================
  Files         139      139              
  Lines       11337    11343       +6     
==========================================
  Hits        10259    10259              
- Misses       1078     1084       +6     
Flag Coverage Δ
total 90.45% <0.00%> (-0.05%) ⬇️
unit 77.54% <0.00%> (-0.05%) ⬇️

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@efaulhaber
efaulhaber requested a review from svchb August 21, 2026 11:55
@efaulhaber
efaulhaber marked this pull request as ready for review August 21, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant