Add pybindings for TextLLMRunner#17481
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/17481
Note: Links to docs will display an error until the docs builds have been completed. ❌ 3 New Failures, 179 PendingAs of commit b88ef54 with merge base cd32e42 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
Hi @NickCao! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at cla@meta.com. Thanks! |
acde4f0 to
b40a829
Compare
|
@pytorchbot label "release notes: llm" |
|
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Meta Open Source project. Thanks! |
|
Hi @NickCao, thanks for the PR. Do check the lint runner check and address it. |
|
This error seems to be valid Take a look? |
b40a829 to
42d535c
Compare
No new invocations of def_property is introduced in this PR? |
|
ok I'll help you merge this |
42d535c to
b88ef54
Compare
Summary
Fixes #12790
Tested in https://github.com/NickCao/executorch-benchmark/tree/text-llm-runner-pr
cc @larryliu0820 @mergennachin @cccclai @helunwencser @jackzhxng