Skip to content

VPython labextension support on remote host. #36

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

Merged
merged 14 commits into from
Oct 25, 2019
Merged

VPython labextension support on remote host. #36

merged 14 commits into from
Oct 25, 2019

Conversation

jcoady
Copy link
Contributor

@jcoady jcoady commented Oct 21, 2019

Made changes to enable the vpython labextension to work with JupyterLab running on a remote host such as binder or JupyterHub.

@jcoady
Copy link
Contributor Author

jcoady commented Oct 21, 2019

Hi Matt

This failed on the Mac build but I only changed 1 line of code in the vpython file with_notebook.py . The other file changes had to do with the vpython labextension which I don't think are part of your build. I think you were working on fixing a Mac build issue with a different pull request and that problem is why this build failed.

John

@mwcraig
Copy link
Contributor

mwcraig commented Oct 24, 2019

I'm going to see if I can reproduce the mac problem tonight.

@mwcraig
Copy link
Contributor

mwcraig commented Oct 24, 2019

Well, I can make the problem happen on my local mac. No idea why it happens yet. What was odd about this is that there wasn't any change in our code that brought this on, so I'm going to check to see if there were some recent conda releases that broke something.

@mwcraig
Copy link
Contributor

mwcraig commented Oct 24, 2019

@jcoady -- I just merged a fix for the test failure (there is something wrong with python 3.7.4 from conda and with conda build 3.8.10 so I'm using earlier versions of each).

Can you rebase this to pull in the latest changes?

If you haven't rebased before or want me to do the rebase and push it to your branch please let me know.

@jcoady
Copy link
Contributor Author

jcoady commented Oct 24, 2019

Hi Matt,

Can you perform the rebase for me and push it to my branch.

Thanks

jcoady added 14 commits October 24, 2019 20:05
For JupyterLab version 1.0.0 and higher we no longer need to copy vpython_data files to appropriate directory. It is now handled in the Jupyter labextension for vpython.
Updated for WebSocket support for vpython running on a remote proxy server.
Updated to support vpython labextension on Jupyterlab version > 1.0.0 .
Updated to support websockets on remote host.
@mwcraig
Copy link
Contributor

mwcraig commented Oct 25, 2019 via email

@jcoady
Copy link
Contributor Author

jcoady commented Oct 25, 2019

Thanks Matt

@jcoady jcoady merged commit d78f92b into vpython:master Oct 25, 2019
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