Skip to content

Commit

Permalink
Fix external kernels
Browse files Browse the repository at this point in the history
  • Loading branch information
davidbrochart committed Jul 24, 2023
1 parent 5373c02 commit b8bef7b
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions plugins/kernels/fps_kernels/routes.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,9 +135,10 @@ async def get_sessions(
):
for session in self.sessions.values():
kernel_id = session.kernel.id
kernel_server = kernels[kernel_id]["server"]
session.kernel.last_activity = kernel_server.last_activity["date"]
session.kernel.execution_state = kernel_server.last_activity["execution_state"]
if kernel_id in kernels:
kernel_server = kernels[kernel_id]["server"]
session.kernel.last_activity = kernel_server.last_activity["date"]
session.kernel.execution_state = kernel_server.last_activity["execution_state"]
return list(self.sessions.values())

async def create_session(
Expand Down Expand Up @@ -307,7 +308,7 @@ async def kernel_channels(
# this is an external kernel
# kernel is already launched, just start a kernel server
kernel_server = KernelServer(
connection_file=kernel_id,
connection_file=self.kernel_id_to_connection_file[kernel_id],
write_connection_file=False,
)
await kernel_server.start(launch_kernel=False)
Expand Down Expand Up @@ -343,15 +344,16 @@ async def process_connection_files(self, changes: Set[Tuple[Change, str]]):
for path, cs in file_changes.items():
for change in cs:
if change == Change.deleted:
if path in kernels:
if path in self.kernel_id_to_connection_file.values():
kernel_id = list(self.kernel_id_to_connection_file.keys())[
list(self.kernel_id_to_connection_file.values()).index(path)
]
del self.kernel_id_to_connection_file[kernel_id]
del kernels[kernel_id]
elif change == Change.added:
try:
data = json.loads(Path(path).read_text())
except BaseException:
except Exception:
continue
if "kernel_name" not in data or "key" not in data:
continue
Expand Down

0 comments on commit b8bef7b

Please sign in to comment.