-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support index is ellipsis for getitem in static mode #32876
Conversation
Thanks for your contribution! |
✅ This PR's description meets the template requirements! |
7238aca
to
87c30fa
Compare
87c30fa
to
cf24451
Compare
Sorry to inform you that cf24451's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
cf24451
to
8b3df16
Compare
8b3df16
to
4874560
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Remove ellipsis
4874560
to
555f0fc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ist/Tensor/None/Ellipsis/bool (#33528) * [cherry-pick 2.1] Polish code for _getitem_impl_ (#32868) * [cherry-pick] Polish code for setitem and getitem (#32911) * [slice getitem] Support getitem idx is Tensor or List (#33000) * [getitem] Support index is None for getitem in static mode (#33001) * [Static getitem] Support static Variable getitem for Ellipsis index (#32876) * [static getitem]Support index is list bool for getitem in static mode (#33298)
PR types
New features
PR changes
APIs
Describe
As the title
For example: