Skip to content

Commit fa3e807

Browse files
authored
Merge pull request #505 from mctoohey/main
Make handle_binary compliant with the KernelMessage.IMessage spec
2 parents a90cf76 + 1237eca commit fa3e807

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/mpl_widget.ts

+6-4
Original file line numberDiff line numberDiff line change
@@ -234,10 +234,12 @@ export class MPLCanvasModel extends DOMWidgetModel {
234234
this.send_draw_message();
235235
}
236236

237-
handle_binary(msg: any, dataviews: any) {
237+
handle_binary(msg: any, buffers: (ArrayBuffer | ArrayBufferView)[]) {
238238
const url_creator = window.URL || window.webkitURL;
239239

240-
const buffer = new Uint8Array(dataviews[0].buffer);
240+
const buffer = new Uint8Array(
241+
ArrayBuffer.isView(buffers[0]) ? buffers[0].buffer : buffers[0]
242+
);
241243
const blob = new Blob([buffer], { type: 'image/png' });
242244
const image_url = url_creator.createObjectURL(blob);
243245

@@ -263,7 +265,7 @@ export class MPLCanvasModel extends DOMWidgetModel {
263265
// button to toggle?
264266
}
265267

266-
on_comm_message(evt: any, dataviews: any) {
268+
on_comm_message(evt: any, buffers: (ArrayBuffer | ArrayBufferView)[]) {
267269
const msg = JSON.parse(evt.data);
268270
const msg_type = msg['type'];
269271
let callback;
@@ -281,7 +283,7 @@ export class MPLCanvasModel extends DOMWidgetModel {
281283
}
282284

283285
if (callback) {
284-
callback(msg, dataviews);
286+
callback(msg, buffers);
285287
}
286288
}
287289

0 commit comments

Comments
 (0)