-
-
Notifications
You must be signed in to change notification settings - Fork 779
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
Strange micropip failure when loading wheel from custom URL #1653
Comments
Well it looks like |
Ok, thanks! So,
While in a Javascript kernel |
Try monkey patching fetch before loading Pyodide: window.orig_fetch = fetch;
window.fetch = async function(...args){
console.log("fetch called with args:", fetch);
let resp;
try {
resp = await orig_fetch(...args);
} catch(e) {
console.log("fetch threw:", e);
throw e;
}
console.log("fetch returned:", resp);
return resp;
} What does it say? |
|
So it did |
And it seems to have thrown a perfectly reasonable error object in response, so I don't know why Pyodide isn't converting it to a valid Python exception... |
Very mysterious. |
Could you provide a reproduction? This behavior seems to suggest a serious bug -- logic errors aside I would hope that it could fail a bit more gracefully.... |
You can try with the URL behind this short link: http://s.here.com/TeGPQR which is publically reachable. |
So I just try to load that wheel on https://pyodide-cdn2.iodide.io/v0.17.0/full/console.html? |
Okay it reproduces thanks. I'll investigate and get back to you. |
Thanks! |
So the problem is that Closed as fixed on main. |
Thanks for your efforts! If I understand correctly you didn't observe any issue on Chrome. I gave it a try, too, using Version 90.0.4430.212 (Official Build) (x86_64) on macOS 10.14.6 and I get his even more impressive traceback below. Can you shed some more light on this, please?
|
Yeah on tip of tree I think there is no problem, on v0.17.0 there is one. Does |
In a JS kernel on Chrome I get for both, |
Hmmmmm. Okay, I think I was never reproducing this then. |
So can you reproduce it in |
Both give me |
No special headers needed. |
So if you use |
For
|
I'm trying to install a soon-to-be-open-sourced package with zero dependencies in JupyterLite like this, but am getting a weird
TypeError: exceptions must derive from BaseException
. Any idea what I can check further?The text was updated successfully, but these errors were encountered: