Yuichiro Tachibana (Tsuchiya)

Results 356 comments of Yuichiro Tachibana (Tsuchiya)

It's about Transformers.js' API spec where you have to pass the `labels` at prediction, not model initialization (see https://huggingface.co/docs/transformers.js/api/pipelines#module_pipelines.ZeroShotClassificationPipeline). So your code should be modified like this: ```python from transformers_js...

Added tests were run successfully. https://github.com/gradio-app/gradio/actions/runs/8980681841/job/24664770460 ``` test/components/test_annotated_image.py .... [ 82%] test/components/test_audio.py ...... [ 83%] test/components/test_bar_plot.py ...... [ 83%] test/components/test_button.py . [ 84%] test/components/test_chatbot.py .. [ 84%] test/components/test_checkbox.py .. [...

Found that this step triggers building the package, including copying `_frontend_code` https://github.com/gradio-app/gradio/blob/72661e3391a432ccd77fb3100e69fbdf95093931/.github/actions/install-all-deps/action.yml#L77-L83 and these are needed by `js/_website/generate_jsons/generate.py` that follows.

Now I think ignoring the [`tool.hatch.build.artifacts` config in `pyproject.toml`](https://github.com/gradio-app/gradio/blob/72661e3391a432ccd77fb3100e69fbdf95093931/pyproject.toml#L72-L79) somehow would be the simplest path, but don't know how...

The functional test failed because this `hatch` started to be needed. I confirmed it would be working after merged into main. Ref: 571823b, https://github.com/gradio-app/gradio/actions/runs/8477748173/job/23229061320?pr=7855

@abidlabs Oh yes it's to be reviewed thanks, but not ready to merge: I want opinions about the comments above.

``` InvalidRequirement: Invalid URL: {root:uri}/../.config/lite-builder  ELIFECYCLE  Command failed with exit code 1.  ELIFECYCLE  Command failed. ERROR: "pybuild:gradio" exited with 1.  ELIFECYCLE  Command failed with exit code 1. /home/runner/work/gradio/gradio/js/app:  ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL  @gradio/[email protected]...

Specifying `packaging>=23.2` doesn't work because `23.2` will be automatically installed by Pyodide as [it ships the version](https://pyodide.org/en/stable/usage/packages-in-pyodide.html) so it causes version conflicts.

Confirmed the built wheel was uploaded and it worked in local.

The functional test failed because this hatch started to be needed. I confirmed it would be working after merged into main. Ref: https://github.com/gradio-app/gradio/commit/571823b4a05f7e41e0b3731d51c5bd86b2e17ddc, https://github.com/gradio-app/gradio/actions/runs/8477748173/job/23229061320?pr=7855