Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Pull request overview
This PR updates the examples and plotting helpers to support switching between GLMakie and CairoMakie via the examples/menu.jl menu, and standardizes plot output handling (default formats, output directory, and filename sanitization).
Changes:
- Add CairoMakie to the examples environment and provide menu options to activate GLMakie or CairoMakie at runtime.
- Make Makie
save_plotchoose a backend-aware default file extension and sanitize output filenames. - Route example outputs to a shared
output/directory and ensure it’s created during installation.
Reviewed changes
Copilot reviewed 12 out of 14 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| ext/VortexStepMethodMakieExt.jl | Backend-aware default data_type, plus filename sanitization for Makie saves. |
| ext/VortexStepMethodControlPlotsExt.jl | Filename sanitization for ControlPlots saves. |
| examples/menu.jl | Adds CairoMakie dependency and menu entries to activate plotting backends. |
| examples/*.jl | Standardize output directory usage and saving toggles across examples. |
| examples/Project.toml | Adds CairoMakie to the examples project. |
| bin/install | Creates output/ directory during installation. |
| docs/src/examples.md | Adds backend comparison documentation. |
| NEWS.md | Notes the new menu backend selection feature. |
| .gitignore | Ignores output directories. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 15 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.