In-situ visualization #0: Make Makie available in 2D VisualizationCallback #2225

benegee · 2025-01-09T10:41:41Z

This is the first step in reintegrating @s6nistam's work on in-situ visualization.

As an alternative to Plots.jl it adds plotting with GLMakie for 2D TreeMesh setups.

github-actions · 2025-01-09T10:41:54Z

Review checklist

This checklist is meant to assist creators of PRs (to let them know what reviewers will typically look for) and reviewers (to guide them in a structured review process). Items do not need to be checked explicitly for a PR to be eligible for merging.

Purpose and scope

The PR has a single goal that is clear from the PR title and/or description.
All code changes represent a single set of modifications that logically belong together.
No more than 500 lines of code are changed or there is no obvious way to split the PR into multiple PRs.

Code quality

The code can be understood easily.
Newly introduced names for variables etc. are self-descriptive and consistent with existing naming conventions.
There are no redundancies that can be removed by simple modularization/refactoring.
There are no leftover debug statements or commented code sections.
The code adheres to our conventions and style guide, and to the Julia guidelines.

Documentation

New functions and types are documented with a docstring or top-level comment.
Relevant publications are referenced in docstrings (see example for formatting).
Inline comments are used to document longer or unusual code sections.
Comments describe intent ("why?") and not just functionality ("what?").
If the PR introduces a significant change or new feature, it is documented in NEWS.md with its PR number.

Testing

The PR passes all tests.
New or modified lines of code are covered by tests.
New or modified tests run in less then 10 seconds.

Performance

There are no type instabilities or memory allocations in performance-critical parts.
If the PR intent is to improve performance, before/after time measurements are posted in the PR.

Verification

The correctness of the code was verified using appropriate tests.
If new equations/methods are added, a convergence test has been run and the results
are posted in the PR.

Created with ❤️ by the Trixi.jl community.

codecov · 2025-01-12T13:13:29Z

Codecov Report

Attention: Patch coverage is 12.82051% with 34 lines in your changes missing coverage. Please review.

Project coverage is 96.79%. Comparing base (72c0571) to head (8568bec).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
ext/TrixiMakieExt.jl	0.00%	29 Missing ⚠️
..._dgsem/elixir_advection_amr_visualization_makie.jl	0.00%	4 Missing ⚠️
src/callbacks_step/visualization.jl	83.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2225      +/-   ##
==========================================
- Coverage   96.88%   96.79%   -0.08%     
==========================================
  Files         490      491       +1     
  Lines       39491    39524      +33     
==========================================
  Hits        38257    38257              
- Misses       1234     1267      +33

Flag	Coverage Δ
unittests	`96.79% <12.82%> (-0.08%)`	⬇️

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

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

sloede · 2025-01-16T12:44:34Z

Nice! Looking forward to this!

benegee · 2025-01-20T11:13:23Z

Ready for review @trixi-framework/developers .

This PR involves places I have not been before (visualization, packages extensions, ...). Any feedback or hints would be highly appreciated.

examples/tree_2d_dgsem/elixir_advection_amr_visualization_makie.jl

DanielDoehring · 2025-01-20T11:18:09Z

src/callbacks_step/visualization.jl

+    # prevents anyone from using other packages, given that appropriate `plot_creator`s are
+    # passed.
+
+    # TODO: rest of this comment?


This should be resolved in the merged source code.

Yes, definitely!
I just need some explanation about the Requires.jl stuff.

@sloede It's been a while since you added this note. Could you help with updating it?

What's the current status - isn't Plots still using Requires.jl? What do we need to update - isn't this comment still valid?

…ee/Trixi.jl into bg/makie-2d-visualization-callback

ranocha

Nice! I just had a quick look. This definitely should get an entry in NEWS.md

ext/TrixiGLMakieExt.jl

ranocha · 2025-01-20T14:04:51Z

src/callbacks_step/visualization.jl

@@ -166,7 +179,7 @@ function (visualization_callback::VisualizationCallback)(integrator)
 end

 """
-    show_plot(plot_data, variable_names;
+    show_plot(visualization_callback, plot_data, variable_names;


There are changes to documented API, so this is breaking. We should either enable the previous behavior again or include this only in a breaking release.

True!
In fact, I did not think about someone calling show_plot directly. For the callback it is only called internally.
Waiting for a breaking release would of course be fine.

I guess nobody is using it actively, but it's documented API.

ranocha · 2025-01-20T14:05:41Z

Project.toml

@@ -55,11 +55,13 @@ UUIDs = "cf7118a7-6976-5b1a-9a39-7adc72f591a4"
 [weakdeps]
 Convex = "f65535da-76fb-5f13-bab9-19810c17039a"
 ECOS = "e2685f51-7e38-5353-a97d-a921fd2c8199"
+GLMakie = "e9467ef8-e4e7-5192-8a1a-b1aee30e663a"


Does it need to be GLMakie.jl, or could it be also one of the other backends?

Having the 3D case in mind, it would be nice if you could interactively turn your plot around.

But I am not sufficiently familiar with Makie. Can you make the implementation independent of a certain backend?

Apparently that is how it is supposed to work. I feel amazed...

I now removed the GLMakie extension and moved its content to our existing Makie extension. For the added elixir both using GLMakie and using CairoMakie work (although the latter might not be too useful).

moved stuff to Makie extension

ranocha · 2025-01-21T12:28:45Z

I converted this to a draft PR so that nobody accidentally merges it.

ranocha · 2025-02-07T08:44:44Z

Shall we include this in the breaking release scheduled for soon? If so, please update the PR accordingly.

s6nistam and others added 6 commits September 23, 2024 09:43

ParaviewCatalyst callback working except PlotData3D

6607023

Merge branch 'main' of github.com:benegee/Trixi.jl

44db6aa

Merge branch 'main' of github.com:benegee/Trixi.jl

88e35f7

Merge branch 'main' of github.com:benegee/Trixi.jl

46feed1

GLMakie 2D VisualizationCallback

d1a6122

delete extra files

034fa5a

benegee added 3 commits January 9, 2025 11:42

delete another file

716352e

wrong comment

a81e8dc

remove leftovers from testing

24fbffe

benegee added 3 commits January 16, 2025 17:48

types for Figure and Axis

d7540dc

use FigureAndAxes struct

9727d3f

format

2f0562b

benegee marked this pull request as ready for review January 20, 2025 11:09

Merge branch 'main' into bg/makie-2d-visualization-callback

5bd1c3a

DanielDoehring requested changes Jan 20, 2025

View reviewed changes

benegee added 3 commits January 20, 2025 12:56

pass plot_arguments to heatmap

4d0e9d3

explain kwargs

dab3337

Merge branch 'bg/makie-2d-visualization-callback' of github.com:beneg…

b5f8b52

…ee/Trixi.jl into bg/makie-2d-visualization-callback

ranocha requested changes Jan 20, 2025

View reviewed changes

benegee added 4 commits January 20, 2025 17:19

snake case

fda5f4f

import what is necessary only

4d401fa

removed GLMakie extensions

1fc1d92

moved stuff to Makie extension

format

72ac1cc

ranocha added visualization breaking labels Jan 21, 2025

ranocha mentioned this pull request Jan 17, 2025

Release v0.10 #2105

Closed

10 tasks

ranocha marked this pull request as draft January 21, 2025 12:28

benegee mentioned this pull request Jan 23, 2025

Contours via Plots.jl are supported for TreeMesh #2230

Open

ranocha mentioned this pull request Feb 7, 2025

Release 0.11 #2265

Open

8 tasks

support for PlotData2DTriangulated

93dd877

benegee changed the title ~~In-situ visualization #0: Make GLMakie available in 2D TreeMesh VisualizationCallback~~ In-situ visualization #0: Make Makie available in 2D VisualizationCallback Feb 11, 2025

benegee added 2 commits February 11, 2025 09:18

Merge branch 'main' of github.com:benegee/Trixi.jl

bf50f02

Merge branch 'main' into bg/makie-2d-visualization-callback

8568bec

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

In-situ visualization #0: Make Makie available in 2D VisualizationCallback #2225

In-situ visualization #0: Make Makie available in 2D VisualizationCallback #2225

benegee commented Jan 9, 2025

github-actions bot commented Jan 9, 2025

codecov bot commented Jan 12, 2025 •

edited

Loading

sloede commented Jan 16, 2025

benegee commented Jan 20, 2025

DanielDoehring Jan 20, 2025

benegee Jan 20, 2025

benegee Jan 21, 2025

sloede Feb 11, 2025

ranocha left a comment

ranocha Jan 20, 2025

benegee Jan 20, 2025

ranocha Jan 21, 2025

ranocha Jan 20, 2025

benegee Jan 20, 2025 •

edited

Loading

benegee Jan 21, 2025

ranocha Jan 21, 2025

ranocha commented Jan 21, 2025

ranocha commented Feb 7, 2025

In-situ visualization #0: Make Makie available in 2D VisualizationCallback #2225

Are you sure you want to change the base?

In-situ visualization #0: Make Makie available in 2D VisualizationCallback #2225

Conversation

benegee commented Jan 9, 2025

github-actions bot commented Jan 9, 2025

Review checklist

Purpose and scope

Code quality

Documentation

Testing

Performance

Verification

codecov bot commented Jan 12, 2025 • edited Loading

Codecov Report

sloede commented Jan 16, 2025

benegee commented Jan 20, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ranocha left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

benegee Jan 20, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ranocha commented Jan 21, 2025

ranocha commented Feb 7, 2025

codecov bot commented Jan 12, 2025 •

edited

Loading

benegee Jan 20, 2025 •

edited

Loading