Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Began stellar streams tutorial #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ojustino
Copy link
Contributor

@ojustino ojustino commented Feb 8, 2019

Here is an adaptation of Mark SubbaRao's stellar streams notebook. It's still a work in progress.

@ojustino ojustino force-pushed the streams branch 2 times, most recently from 9ccef60 to ee0035d Compare March 19, 2019 14:26
@pkgw
Copy link
Contributor

pkgw commented Mar 25, 2019

It looks like this is blocked until the timeseries stuff moves along, but in the meantime — when I run the wwt.set_view('solar system') command, nothing changes in my WWT widget. Am I using a too-old version of something?

@ojustino
Copy link
Contributor Author

ojustino commented Mar 25, 2019

When this happens for me, it's due to the SSL issues. After you ran the second cell (wwt = WWTJupyterWidget() and so on), did you click the lock icon in the address bar and select "Disable protection for now?" Unfortunately, you have to do it each time you load the page on Binder.

Actually, I'm getting the same behavior as well, even when I disable HTTPS. I'm not quite sure what's going on at the moment.

@pkgw
Copy link
Contributor

pkgw commented Mar 26, 2019

@astrofrog Any insight on the set_view('solar system') issue I mentioned above?

@ojustino
Copy link
Contributor Author

ojustino commented Apr 1, 2019

@astrofrog @pkgw Below, I've pasted the output I get in the console when I try to perform any action with the wwt object that makes a call to the SDK. It looks like the connection to wwt_json_api.js is broken; wwt_apply_json_message is the function that receives a message from Python and translates it to one the WWT SDK can recognize. Do we know why Jupyter isn't recognizing it anymore?

Exception in Comm callback TypeError: "this.wwt_window.wwt_apply_json_message is not a function"
    handle_custom_message wwt.js:122
    Backbone 4
    _handle_comm_msg widget.js:213
    _callback comm.js:194
    handle_msg comm.js:202
    CommManager.prototype.comm_msg/this.comms[content.comm_id]< comm.js:132
 handle_custom_message@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/nbextensions/pywwt/index.js?v=20190401205957:32530:7
triggerEvents@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/nbextensions/jupyter-js-widgets/extension.js?v=20190401205957:26514:32
triggerApi@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/nbextensions/jupyter-js-widgets/extension.js?v=20190401205957:26500:19
eventsApi@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/nbextensions/jupyter-js-widgets/extension.js?v=20190401205957:26299:16
Events.trigger@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/nbextensions/jupyter-js-widgets/extension.js?v=20190401205957:26490:5
WidgetModel</WidgetModel.prototype._handle_comm_msg@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/nbextensions/jupyter-js-widgets/extension.js?v=20190401205957:30248:17
Comm.prototype._callback@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/static/notebook/js/main.min.js?v=0cd6a0ce26767cc779cc03d1661bf989:36728:24
Comm.prototype.handle_msg@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/static/notebook/js/main.min.js?v=0cd6a0ce26767cc779cc03d1661bf989:36736:16
CommManager.prototype.comm_msg/this.comms[content.comm_id]<@https://hub.mybinder.org/user/ojustino-pywwt-notebooks-al3v7zhg/static/notebook/js/main.min.js?v=0cd6a0ce26767cc779cc03d1661bf989:36666:37

@pkgw
Copy link
Contributor

pkgw commented Apr 1, 2019

@ojustino That's what I was seeing as a result of issue WorldWideTelescope/pywwt#191, which should be fixed by PR WorldWideTelescope/pywwt#192, which is blocked on the issue with black images in the CI.

For the purposes of this PR, I think you should be able to get going again by temporarily downgrading the notebook package to version 5.7.5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants