Skip to content
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

BUG: fix sinc for torch #56

Merged
merged 1 commit into from
Dec 9, 2024
Merged

BUG: fix sinc for torch #56

merged 1 commit into from
Dec 9, 2024

Conversation

lucascolley
Copy link
Collaborator

@lucascolley lucascolley marked this pull request as ready for review December 9, 2024 18:06
@lucascolley lucascolley merged commit ee12cd1 into data-apis:main Dec 9, 2024
10 checks passed
@lucascolley
Copy link
Collaborator Author

@ev-br I'll publish a release with this fix for you once @crusaderky has a PR up to SciPy to accommodate co-vendoring this repo and array-api-compat

@lucascolley lucascolley deleted the gh49 branch December 9, 2024 18:08
@crusaderky
Copy link
Contributor

@ev-br I'll publish a release with this fix for you once @crusaderky has a PR up to SciPy to accommodate co-vendoring this repo and array-api-compat

Shouldn't the release of array-api-extra happen before the PR in scipy?
The PR in scipy should change the tagged version of the array_api_extra submodule, if I understand correctly.

@lucascolley
Copy link
Collaborator Author

Shouldn't the release of array-api-extra happen before the PR in scipy?

Before the SciPy PR is merged, yes. But it may be useful to give the PR a go first and catch any bugs here before release.

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

Successfully merging this pull request may close these issues.

BUG: sinc: broken on torch
2 participants