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

MAINT, BUG: sync with upstream to unconditionaly disable use of aligned_alloc #3

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

tylerjereddy
Copy link

@tylerjereddy
Copy link
Author

I'm going to self-merge, trivial sync to upstream needed for release.

@tylerjereddy tylerjereddy merged commit 9367142 into scipy:cpp Mar 25, 2024
@tylerjereddy tylerjereddy deleted the treddy_disable_aligned_alloc branch March 25, 2024 21:46
tylerjereddy added a commit to tylerjereddy/scipy that referenced this pull request Mar 25, 2024
* Fixes scipygh-20300 by syncing `pocketfft` again, this time
to completely disable `aligned_alloc`; see scipy/pocketfft#3
for details, but in short our more conservative shim
was not sufficient for `conda-forge`, so let's just do the same
thing NumPy did...

[skip cirrus] [skip circle]
j-bowhay pushed a commit to scipy/scipy that referenced this pull request Mar 25, 2024
* Fixes gh-20300 by syncing `pocketfft` again, this time
to completely disable `aligned_alloc`; see scipy/pocketfft#3
for details, but in short our more conservative shim
was not sufficient for `conda-forge`, so let's just do the same
thing NumPy did...
@rgommers
Copy link
Member

rgommers commented Apr 1, 2024

@tylerjereddy @lucascolley I think this is starting to go in the wrong direction. This repo is meant to be an exact mirror of upstream (EDIT: see scipy/scipy#19882), so we shouldn't see any diverging commit hashes.

I'm thinking of doing the following:

  • push a new branch containing the diverging commits to preserve them
  • bring the default cpp branch back in sync with the upstream cpp branch
  • disabling pull requests and setting branch protection such that only regular pushes to the default branch are possible.

@tylerjereddy
Copy link
Author

fair enough

tylerjereddy added a commit to tylerjereddy/scipy that referenced this pull request May 2, 2024
* Fixes scipygh-20300 by syncing `pocketfft` again, this time
to completely disable `aligned_alloc`; see scipy/pocketfft#3
for details, but in short our more conservative shim
was not sufficient for `conda-forge`, so let's just do the same
thing NumPy did...

[skip cirrus] [skip circle]
@lucascolley
Copy link
Member

I'm thinking of doing the following:

push a new branch containing the diverging commits to preserve them
bring the default cpp branch back in sync with the upstream cpp branch
disabling pull requests and setting branch protection such that only regular pushes to the default branch are possible.

Would you like to do this now? Let me know if I can help

@rgommers
Copy link
Member

Would like to, but realistically not going to any time soon (too many other higher-prio things pending). If you'd want to do this, that would be great! As long as all the commits remain on a branch in the repo, this should be safe enough. Let me know if you want to go ahead and need more permissions.

@lucascolley
Copy link
Member

Would like to, but realistically not going to any time soon (too many other higher-prio things pending). If you'd want to do this, that would be great! As long as all the commits remain on a branch in the repo, this should be safe enough. Let me know if you want to go ahead and need more permissions.

I'm happy to! And yes, I think I need more permissions.

@rgommers
Copy link
Member

And yes, I think I need more permissions.

done (... full access to the project, including sensitive and destructive actions ...) - with great power and all that:)

@lucascolley
Copy link
Member

@rgommers I've pushed a new branch commit-archive, but I am unable to force push to the cpp branch due to GH006: Protected branch update failed for refs/heads/cpp

@rgommers
Copy link
Member

I'll have a look. Is https://github.com/lucascolley/pocketfft/tree/cpp up-to-date?

@lucascolley
Copy link
Member

I think I've figured out how to disable the branch protection temporarily

@rgommers
Copy link
Member

Good - yes you should be able to.

@lucascolley
Copy link
Member

scipy:cpp now matches mreineck:cpp. I'm not sure how to disable pull requests, though

@rgommers
Copy link
Member

Great, thanks! I don't think pull requests can be completely disabled, but I edited the branch protection rule to require 3 approvals, as a strong signal that PRs shouldn't be merged. Will open an issue to describe the state of things.

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.

4 participants