-
Notifications
You must be signed in to change notification settings - Fork 7
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
Model class 'IdomModel' from module 'idom-client-jupyter' is loaded but can not be instantiated #24
Comments
Thanks for reporting this. There's been a lot of changes recently to idom and not all of them have made their way into idom-jupyter just yet. I suspect that's probably the cause. |
@rmorshea thanks for the quick response. very strange behavior here. |
That's very helpful information. Would have been very hard to find that out on my own. If you open the debug console on your browser, do you get any interesting tracebacks? |
no |
Maybe related: jupyterhub/mybinder.org-deploy#2407 |
For some reason when I launch through https://mybinder.org I end up with instances deployed at https://hub-binder.mybinder.ovh. However, when I deploy via https://ovh2.mybinder.org/ I get redirected to https://hub.ovh2.mybinder.org which, as you explain, results in errors. |
I have a draft PR to upgrade to the latest widget template and I see similar errors when I deploy that branch, even via https://mybinder.org. Hard to say exactly what's going on at the moment. Will keep digging when I find time though. |
even when i use the same link |
I'm unable to reproduce in ovh2 now. I'm also unable to reproduce when testing this out with repo2docker. I'm not really sure what to do here at the moment given that I'm no longer able to reproduce. Will close for now, but please comment if this persists. |
@rmorshea can you verify this for me? thanks. |
Ok, I just double checked and I think I was looking at my open PR when testing things out. I can confirm that |
I'll merge that PR and cut a release shortly. |
Ok, the PR has been merged. It would be great to get confirmation that everything works on your end before I do a release. |
yes, can confirm it works even for my forked version of this repo now. thanks. |
|
Launching binder notebook using the link from README results in the following error after executing cells.
build logs:
The text was updated successfully, but these errors were encountered: