executorch
executorch copied to clipboard
Clean up llava deps and consolidate all HF deps
python3 -m examples.portable.scripts.export --model_name="llava" works fine on my mac M1. No strict mode is needed. And can directly working on the most recent transformers lib.
It seems like the old restriction about numpy and transformers are not longer true. If CI gives green light we can cleanup the stale setup for llava and use the root setup (executorch/install_requirements.sh) as the source of truth
:link: Helpful Links
:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/4320
- :page_facing_up: Preview Python docs built from this PR
Note: Links to docs will display an error until the docs builds have been completed.
:x: 1 New Failure, 1 Cancelled Job
As of commit a89f1cf2927f5235672dc26fe2acfb78824c9061 with merge base 39aeff99dcb4d320fed090101cf5f95c0be1bcba ():
NEW FAILURE - The following job has failed:
- pull / test-llama-runner-qnn-linux (fp32, cmake, qnn) / linux-job (gh)
RuntimeError: Command docker exec -t d701f74bff162e0a074f3dcc9159a96ccd255541773232ed480c74a2be94aafd /exec failed with exit code 2
CANCELLED JOB - The following job was cancelled. Please retry:
- trunk / test-models-macos (cmake, vit, xnnpack-delegation, macos-m1-stable, 90) / macos-job (gh)
##[error]The operation was canceled.
This comment was automatically generated by Dr. CI and updates every 15 minutes.
@guangy10 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.
It works now on my Linux devserver, but not in the CI due to OOM on the runner.
@larryliu0820 could you help review this PR? I'd like to consolidate to one 🤗 transformers versions for ET. That will make future model enablement and benchmarking much easier
@larryliu0820 could you help review this PR? I'd like to consolidate to one 🤗
transformersversions for ET. That will make future model enablement and benchmarking much easier
Can you please rebase? I just cleaned up the dependencies for llava.
@guangy10 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.