Skip to content

Commit 11560bc

Browse files
Update Makie plot recipe kwargs for EnsembleSummary
`EnsembleSummary`'s plot recipe refered to the variable indices as `trajectories`, likely typo from the `EnsembleSimulation` recipe above where that nomenclature made sense. Fixed it to use `idxs`, allowing users to select which variables to plot the summaries of, in line with the documentation.
1 parent bbdd876 commit 11560bc

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

ext/SciMLBaseMakieExt.jl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,7 @@ end
344344
# TODO: should `error_style` be Makie plot types instead? I.e. `Band`, `Errorbar`, etc
345345
function Makie.convert_arguments(::Type{<:Lines},
346346
sim::SciMLBase.EnsembleSummary;
347-
trajectories = sim.u.u[1] isa AbstractArray ? eachindex(sim.u.u[1]) :
347+
idxs = sim.u.u[1] isa AbstractArray ? eachindex(sim.u.u[1]) :
348348
1,
349349
error_style = :ribbon, ci_type = :quantile,
350350
kwargs...)
@@ -384,7 +384,7 @@ function Makie.convert_arguments(::Type{<:Lines},
384384

385385
makie_plotlist = Makie.PlotSpec[]
386386

387-
for (count, idx) in enumerate(trajectories)
387+
for (count, idx) in enumerate(idxs)
388388
push!(makie_plotlist,
389389
S.Lines(sim.t, u[idx]; color = Makie.Cycled(count), label = "u[$idx]"))
390390
if error_style == :ribbon

0 commit comments

Comments
 (0)