Gyeongjae Choi

Results 674 comments of Gyeongjae Choi

Yep, +1 for fixing the warehouse bug. Let me see if I can understand and modify the warehouse codebase.

If there are some changes in Pyodide and if you want to use those features in the micropip, you are welcome to do so. We try to keep the compatibility...

Yes, I guess removing the patch and setting [`LIBHEIF_ROOT`](https://github.com/bigcat88/pillow_heif/blob/6b3e0492ab7ef80a8929b457e50e05e630526a06/setup.py#L83) to WASM_LIBRARY_DIR would work

@agriyakhetarpal Thanks for the detailed information. Let me take a look, and yes, packages with libraries are quite tricky to build when they use no isolation option. If this is...

Opened a PR (#5178). Ideally, in the future we unvendor recipes, each package may use a different version of shared libraries.

Superseded by #5178. Thanks for investigating the issue Agriya!

Maybe we can allow putting extra suffix for `emfs://` or `file://` protocols only? Then people might be able to download the package first, and install it from the file system.

> The [micropip documentation](https://micropip.pyodide.org/en/stable/project/usage.html) also lists a syntax await micropip.install("snowballstemmer @ https://.../snowballstemmer.*.whl") Huh, I think that we never supported this syntax.

I prefer the latter (`micropip.install("….whl", features=["all"])`).

Thanks for bringing this up, @juntyr! Reducing the number of shared libraries would help reduce loading time and memory usage. However, as you mentioned, this is up to the package...