Skip to content

Commit

Permalink
Fix stability issues caused by Process Status tab (#111)
Browse files Browse the repository at this point in the history
  • Loading branch information
fonsp authored Apr 27, 2023
1 parent ec8bf9a commit 9b6870d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/Actions.jl
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@ function process(
notebook = Pluto.SessionActions.open(server_session, abs_path; run_async=false)
# get the state object
original_state = Pluto.notebook_to_js(notebook)
delete!(original_state, "status_tree")
# shut down the notebook
if !keep_running
@info "Shutting down notebook process" s.path
Expand Down
6 changes: 4 additions & 2 deletions src/HTTPRouter.jl
Original file line number Diff line number Diff line change
Expand Up @@ -143,8 +143,10 @@ function make_router(
new["cell_results"] = filter(state["cell_results"]) do (id, cell_state)
id ids_of_cells_that_ran
end
# ... nothing about bond values, because we don't want to synchronize among clients.
new["bonds"] = Dict{String,Dict{String,Any}}()
# ... nothing about bond values, because we don't want to synchronize among clients. and...
delete!(new, "bonds")
# ... we ignore changes to the status tree caused by a running bonds.
delete!(new, "status_tree")
new
end

Expand Down
2 changes: 1 addition & 1 deletion test/runtestserver.jl
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
using PlutoSliderServer
ENV["JULIA_DEBUG"] = PlutoSliderServer

import Random

Random.seed!(time_ns())
test_dir = tempname(cleanup=false)
Expand Down

0 comments on commit 9b6870d

Please sign in to comment.