Closed mtsokol closed 9 months ago
Merging #627 (5bd29ad) into main (82fb0d5) will decrease coverage by
0.38%
. The diff coverage is58.57%
.
Ok, I think I addressed all comments - the PR is ready for another review.
LGTM, feel free to hit the button. 😄
Thanks! Unfortunately I see: "You’re not authorized to merge this pull request."
LGTM, feel free to hit the button. 😄
Thanks! Unfortunately I see: "You’re not authorized to merge this pull request."
What about now?
Merged! Thank you!
Hi @hameerabbasi,
This PR adds
sort
andtake
Array API functions for the COO format to the main namespace.sort
function, where looping happens, is numba jitted.