pytorch / executorch

On-device AI across mobile, embedded and edge for PyTorch
https://pytorch.org/executorch/
Other
2.14k stars 353 forks source link

[CI] llava_encoder CI on PRs does not use pinned pytorch commit #3940

Open clee2000 opened 5 months ago

clee2000 commented 5 months ago

test-models-linux (cmake, llava_encoder, portable, linux.4xlarge, 90) / linux-job CI job does not use pinned pytorch commit in CI

Example: https://github.com/pytorch/executorch/actions/runs/9472225169/job/26099055416

It reinstalls pytorch from whatever version is specified in install_requirements: https://github.com/pytorch/executorch/blob/dcf00b776abaaec81fb34c66521db8ea682ba72e/examples/models/llava_encoder/install_requirements.sh#L24

Is this expected?

digantdesai commented 5 months ago

cc @iseeyuan - can you help here?