Hood Chatham
Hood Chatham
Ah that's a cool api, thanks for the reference @coatless. I really like the look of `myShelter.purge();`, since collecting individual references and destroying them is a pain. Especially in our...
If you work on this, we will answer your questions.
The problem is that the .so preloading logic is different in the Pyodide virtual environment. Instead of using `dynload.ts`: https://github.com/pyodide/pyodide/blob/main/src/js/dynload.ts it uses this much crappier custom implementation: https://github.com/pyodide/pyodide/blob/main/src/templates/python#L125-L145 It would...
What happens if you clone it and run `python -m build .` in the source directory? Presumably you're saying that doesn't work?
Looks like the sdist at least includes a `setup.py`, so being optimistic, maybe they already did the extra weird steps before making the sdist and it won't be a problem?
With the node runner: ```js > resolveGlobalSymbol("s_cmp") < {sym: undefined, name: 's_cmp'} > LDSO.loadedLibsByName["libopenblas.so"].exports.s_cmp < ƒ $s_cmp() { [native code] } ``` vs in normal Pyodide: ```js > resolveGlobalSymbol("s_cmp") <...
@ryanking13 can we decide whether `forceGlobal` should be true based on whether the dynamic library is not installed in `site-packages`? So that it's loaded globally if it's in `neededDylibs` or...
I'm actually not sure why `libopenblas.so` is loaded at all since the code makes it look like we only load libs under `site-packages` and it's installed at `.venv-pyodide/lib/libopenblas.so`...
Ah right it's because `defaultLdLibraryPath` includes `.venv-pyodide/lib/`
I think what we need is this patch to Emscripten. This should also allow us to strip a lot of the stuff out of `dynload.ts`. ```patch --- a/src/library_dylink.js +++ b/src/library_dylink.js...