Send WebAssembly binary over Jupyter WebSocket#461
Open
kylebarron wants to merge 3 commits intomainfrom
Open
Conversation
|
one suggestion, you can use some indirection to create a model with just the static contents so that it is hoisted from each model instance: import ipywidgets
import anywidget
import traitlets
class StaticAsset(ipywidgets.Widget):
contents = traitlets.Any().tag(sync=True)
asset = StaticAsset(contents=b"hello, world")
class Widget(anywidget.AnyWidget):
_esm = """
async function load_asset(model, name) {
let model_id = model.get(name).slice("IPY_MODEL_".length);
let asset_model = await model.widget_manager.get_model(model_id);
return asset_model.get("contents");
}
async function render({ model, el }) {
let asset = await load_asset(model, "asset")
el.innerText = new TextDecoder().decode(asset)
}
export default { render }
"""
asset = traitlets.Any(asset).tag(sync=True, **ipywidgets.widget_serialization)
Widget()Each widget instance will just have |
|
It should be noted this is just a way to get the static assets into the front end, but derived objects (e.g., initialized parquet module) would need to be cached somewhere. Probably easiest to make some global for now, but would be more elegant if anywidget could provide a more ideomatic API. Concretely, import ipywidgets
import anywidget
import traitlets
class StaticAsset(ipywidgets.Widget):
contents = traitlets.Any().tag(sync=True)
asset = StaticAsset(contents=b"hello, world")
class Widget(anywidget.AnyWidget):
_esm = """
async function load_asset(model, name) {
let model_id = model.get(name).slice("IPY_MODEL_".length);
let asset_model = await model.widget_manager.get_model(model_id);
return asset_model.get("contents");
}
async function initialize({ model }) {
if (!globalThis._TREVORS_DECODED_ASSET) {
// cache this globally for all others....
let asset = await load_asset(model, "asset");
globalThis._TREVORS_DECODED_ASSET = new TextDecoder().decode(asset);
}
}
async function render({ model, el }) {
el.innerText = globalThis._TREVORS_DECODED_ASSET
}
export default { initialize, render }
"""
asset = traitlets.Any(asset).tag(sync=True, **ipywidgets.widget_serialization)
Widget() |
eae5d2e to
c6c2172
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
We use Parquet as the internal format for data transfer. For reasons linked, Parquet is great. But to read Parquet on the client, we need to use a Wasm-based Parquet reader like my own https://github.com/kylebarron/parquet-wasm. Wasm-based libraries need a sidecar binary
.wasmfile, which is usually distributed separately.We currently fetch this file from CDN, but for environments with a strong outbound firewall, a CDN may not be allowed. See #457. To get around this, we serialize the gzipped Wasm content on the Anywidget model itself. We then decompress it on the client and pass it into Parquet-wasm's initializer.
Closes #457