Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion rendercanvas/anywidget.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
from .base import BaseCanvasGroup, BaseRenderCanvas, logger
from .asyncio import loop
from .core.encoders import encode_array, CAN_JPEG
from .core.events import valid_event_types

import numpy as np
import anywidget
Expand Down Expand Up @@ -156,7 +157,7 @@ def _rfb_handle_msg(self, widget, content, buffers):
)
elif event_type == "close":
self.close()
else:
elif event_type in valid_event_types:
# Compatibility between new renderview event spec and current rendercanvas/pygfx events
event["event_type"] = event.pop("type")
event["time_stamp"] = event.pop("timestamp")
Expand Down
5 changes: 2 additions & 3 deletions rendercanvas/jupyter.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
import time

from .base import BaseCanvasGroup, BaseRenderCanvas
from .core.events import EventType
from .core.events import valid_event_types
from .asyncio import loop

import numpy as np
Expand Down Expand Up @@ -39,7 +39,6 @@ def __init__(self, *args, **kwargs):
self._last_image = None
self._is_closed = False
self._draw_request_time = 0
self._rendercanvas_event_types = set(EventType)

# The send_frame() method was added in jupyter_rfb 1.0, but it was always there as a private method,
# so we can make it backwards compatible.
Expand Down Expand Up @@ -129,7 +128,7 @@ def handle_event(self, event):

# Only submit events that rendercanvas knows. Otherwise, if new events are added
# to jupyter_rfb that rendercanvas does not (yet) know, rendercanvas will complain.
if event_type in self._rendercanvas_event_types:
if event_type in valid_event_types:
self.submit_event(event)


Expand Down
5 changes: 5 additions & 0 deletions rendercanvas/pyodide.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@

from .base import BaseRenderCanvas, BaseCanvasGroup
from .asyncio import loop
from .core.events import valid_event_types

if "pyodide" not in sys.modules:
raise ImportError("This module is only for use with Pyodide in the browser.")
Expand Down Expand Up @@ -87,6 +88,10 @@ def _on_event(self, event):
# Called from JS
event = event.to_py()

# renderview.js may generate events that rendercanvas does not yet know
if event["type"] not in valid_event_types:
return

# Compatibility between new renderview event spec and current rendercanvas/pygfx events
event["event_type"] = event.pop("type")
event["time_stamp"] = event.pop("timestamp")
Expand Down
Loading