Skip to content

[5.0.x] Add missing memcpy header #13297

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

Open
wants to merge 1 commit into
base: v5.0.x
Choose a base branch
from

Conversation

Yiltan
Copy link
Contributor

@Yiltan Yiltan commented Jun 9, 2025

Cherry picked from #13295

(commit aa55774)

Signed-off-by: Yiltan Hassan Temucin <yiltan.temucin@amd.com>

(cherry picked from commit aa55774)
@github-actions github-actions bot added this to the v5.0.8 milestone Jun 9, 2025
@hppritcha
Copy link
Member

something is broken with MPI4PY. The error is almost certainly not related to changes made in this PR.

@hppritcha
Copy link
Member

@dalcinl would you have suggestions on debugging what's going on with mpi4py? it looks like some kind of problem with generation of mpi4py internal docs?

@dalcinl
Copy link
Contributor

dalcinl commented Jun 10, 2025

@hppritcha Sorry for the late reply, I've been offline for a few days. I need a couple extra days to take a look about what is going on. It is most likely related to a change in latest Cython release introducing a regression on how Python docstrings/method signatures are generated.

@dalcinl
Copy link
Contributor

dalcinl commented Jun 10, 2025

@hppritcha I pushed a quick fix to mpi4py/mpi4py@master. Can you try restarting the workflow here?

@jsquyres
Copy link
Member

@dalcinl Worked -- thanks!

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.

5 participants