Sync overlays through traits#151
Conversation
3fdcb10 to
0013030
Compare
0013030 to
5797384
Compare
2270b2b to
4e887fc
Compare
|
It works. My only remarks are:
|
I am not sure if this can cause a real issue in practice. I think we should test loading a big MOC, fits image or catalog votable that is currently well handled in master and see if it is not in this branch. If there are problems we could maybe try evaluating the size of the overlay and send an event or not depending on it.
I think we should do that in another PR maybe ? on the JS side one can call https://cds-astro.github.io/aladin-lite/Aladin.html#removeOverlay with the name of overlay to remove. |
7a69736 to
f951982
Compare
3a46bde to
2fd2057
Compare
* fix the documentation examples. overlays do correctly appear on the generated HTML static file because is its passed with traitlets instead of being sent with events. * if the user want to export the aladin view to HTML, using embed_minimal_html from ipywidgets do recognize the add of overlays * add of colormap traitlet to replace the old event * bind projection change with the change on the python side (aladin.projection)
273187e to
c2cfd64
Compare
This is based on the first cleanup done in #150
This contains:
current online documentation (no overlays are showed):
now:
embed_minimal_htmlfrom ipywidgets does recognize the add of overlaysTo export the aladin object to html through ipywidgets:
This gives this html file with the table overlays correctly contained in it:
