-
-
Notifications
You must be signed in to change notification settings - Fork 308
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
display fixes #3057
display fixes #3057
Conversation
Compile Times benchmarkNote, that these numbers may fluctuate on the CI servers, so take them with a grain of salt. All benchmark results are based on the mean time and negative percent mean faster than the base branch. Note, that GLMakie + WGLMakie run on an emulated GPU, so the runtime benchmark is much slower. Results are from running: using_time = @ctime using Backend
# Compile time
create_time = @ctime fig = scatter(1:4; color=1:4, colormap=:turbo, markersize=20, visible=true)
display_time = @ctime Makie.colorbuffer(display(fig))
# Runtime
create_time = @benchmark fig = scatter(1:4; color=1:4, colormap=:turbo, markersize=20, visible=true)
display_time = @benchmark Makie.colorbuffer(display(fig))
|
What about changing We could also think about having a slightly more "convenient" syntax for opening new windows with this. Instead of |
I would also suggest adding |
Yeah I think those are good ideas, but right now I just want to make a small bug fix PR. |
So, GLMakie not opening a window with activated Plotpane really was not a great idea...
Lots of complaints, and I also tripped over this quite a bit.
This PR changes this back to opening a window by default for GLMakie!
But this means, that one now needs to call
GLMakie.activate!(inline=true)
to get GLMakie back into the plotpane.This PR also changes a few other things:
can_show_inline(backend)
was weirdly buggy with VSCode, a.k.a most of the time it worked that e.g. WGLMakie would show in the plotpane if the plotpane is enabled, but sometimes it just stopped working...Not sure why, but looking at the VSCode code, it should never work the waycan_show_inline
is implemented (not checking forMIME"juliavscode/html"
) - but it definitely worked correctly most of the time 🤷display(plot; inline=false)
to force a window even if inline is true. Of course this doesn't work the other way around, since VSCode somehow doesn't get into the show path anymore with callingdisplay(fig; inline=true)
directly... Not sure why...