Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

lucyleeow
Copy link
Member

Reference Issues/PRs

Towards #26024

What does this implement/fix? Explain your changes.

Add array API support to precision_recall_curve

Any other comments?

Copy link

github-actions bot commented Sep 22, 2025

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 145ba36. Link to the linter CI: here

@lucyleeow
Copy link
Member Author

lucyleeow commented Sep 27, 2025

Tests fail on torch because it does not support negative indexing (data-apis/array-api-compat#144, pytorch/pytorch#59786) - it seems we could use flip but that creates a copy...?

Seems unlikely to change?

And from previous interactions with pytorch devs, my impression was that the assumption that slice steps are non-negative is baked too deep into the torch core to really hope it'll get fixed in torch itself.

Not sure if there is a standard workaround for this? Or do we need to re-write to avoid negative indexing? Or skip for torch...?

Edit: scipy uses flip, even though it creates a copy, but shouldn't be too bad in this case, and does not seem like there is anything else we can do so 🤷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant