Skip to content

Commit 22a2e8b

Browse files
timblakelycopybara-github
authored andcommitted
Fix a weird bug where using FFN inference interactively in Colab caused a kernel crash (!) when subsequent inference calls are made.
PiperOrigin-RevId: 807795049
1 parent 2b3a67d commit 22a2e8b

1 file changed

Lines changed: 8 additions & 3 deletions

File tree

ffn/inference/executor.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -294,9 +294,14 @@ def _run_executor(self):
294294
self.active_clients += 1
295295
logging.info('client %d starting', client_id)
296296
else:
297-
self.registered_clients.remove(-client_id - 1)
298-
logging.info('client %d terminating', -client_id - 1)
299-
self.active_clients -= 1
297+
try:
298+
self.registered_clients.remove(-client_id - 1)
299+
logging.info('client %d terminating', -client_id - 1)
300+
self.active_clients -= 1
301+
except KeyError:
302+
logging.warning(
303+
'client %d not known or already terminated', -client_id - 1
304+
)
300305
else:
301306
client_id, seed, image, fetches = data
302307
l = len(ready)

0 commit comments

Comments
 (0)