search

ipython / ipykernel

IPython Kernel for Jupyter
https://ipykernel.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
654 stars 370 forks source link

Comm target_module #252

Open SylvainCorlay opened 7 years ago

SylvainCorlay commented 7 years ago

The comms have a target_module trait that is not documented in the spec and not used by ipywidgets.

takluyver commented 6 years ago

This identifies a requirejs module to load the target from. The idea was that either side could specify a fully qualified target name for the function to run on the other side.

I don't know of any code using target_module, but mobilechelonian uses view_module at the widgets level, which works in a similar way.

SylvainCorlay commented 6 years ago

yes, view_module and model_module are ipywidgets things, but target_module seems to be some left-over from older code, probably a fossil of the pre-big-split (tm) era.

SylvainCorlay commented 6 years ago

cc @jasongrout

takluyver commented 6 years ago

I'd assume it was added at the same time as view_module and model_module, for symmetry.