Skip to content

Conversation

DannyYuyang-quic
Copy link
Contributor

Summary

Fix broken unpacking in T5 dataset loading

Test plan

python backends/qualcomm/tests/test_qnn_delegate.py TestExampleOssScript.test_t5 -s ${device_id} -m ${soc} --build_folder build-android/ --executorch_root . --artifact_dir . --qa_dataset ${path_to_SQuAD-v1.1.csv}

Copy link

pytorch-bot bot commented Aug 25, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/13625

Note: Links to docs will display an error until the docs builds have been completed.

❌ 1 New Failure, 1 Unrelated Failure

As of commit 19eb419 with merge base def4c64 (image):

NEW FAILURE - The following job has failed:

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 25, 2025
@DannyYuyang-quic
Copy link
Contributor Author

@pytorchbot label "release notes: qualcomm"

@pytorch-bot pytorch-bot bot added the release notes: qualcomm Changes to the Qualcomm backend delegate label Aug 25, 2025
Copy link
Contributor

@cccclai cccclai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix

@cccclai cccclai merged commit 3f483df into pytorch:main Aug 25, 2025
103 of 106 checks passed
agrima1304 pushed a commit to agrima1304/executorch that referenced this pull request Aug 26, 2025
pytorch#13625)

### Summary
Fix broken unpacking in T5 dataset loading

### Test plan
``` bash 
python backends/qualcomm/tests/test_qnn_delegate.py TestExampleOssScript.test_t5 -s ${device_id} -m ${soc} --build_folder build-android/ --executorch_root . --artifact_dir . --qa_dataset ${path_to_SQuAD-v1.1.csv}
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: qualcomm Changes to the Qualcomm backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants