-
Notifications
You must be signed in to change notification settings - Fork 4
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
COMM_WORLD.Get_size() returns 1 reguardless of num threads launched on Github Runner #13
Comments
There is nothing I can do about it. The root issue is the broken MPICH package in Ubuntu 24.04, six month after and the issue is still unresolved. IIRC, the required fix is available in 24.10, but it has not been backported to the 24.04 LTS release. This issue effectively renders MPICH unusable on Ubuntu 22.04. The only thing you can do for the time being is replacing |
I wish this were advertised on the README while the bug is active, or that the setup script issued a warning about it - or even that this issue wasn't closed! Quite nasty to be bitten by one's MPI job silently duplicating. |
@TysonRayJones I usually close issues about things that a) I'm not responsible off, and b) I do not plan to take active action. Generally speaking, the issue trackers in my projects are to help me to organize my (for free) open-source work, not to help others with their own business. 90% of my open-source work is related to dealing with third-party bugs, and I'm sick of it. I wish people spend their time contributing towards solutions, rather than dropping very politically correct and nicely written complaints about stuff I done for free in my spare time I should rather dedicate to my family. Once again, I must clarify: All the work I do in the mpi4py organization is for my own amusement and enlightenment. I have no obligations to any individual or organization. When you use open-source software written by individuals like me, you definitely get more value than what you pay for, so entitled complaints are totally out place. Those who do not like these simple facts are most welcome to fork my BSD-license repositories and adapt things to whatever workflow fit their needs, and even make everything open-source again for the benefit of the world and deal with the maintenance nightmares. All that being said, I'm reopening this issue. I'm annoyed enough about this bug that will try to do something about it on my side for the need of my projects. I've been working on it during the last couple weeks, still deciding where to install MPICH from a Ubuntu PPA from a newer release, or use a conda-forge build. There is no ETA, though. |
@dalcinl I totally get the sentiment and your frustration is justified - sincere apologies if I've contributed to it! Your GA is super useful, hence its integration in other free and open-source work, and its role in a community of developers trying to help one another (in which you may be a heavy-lifter!). I agree there is no place for entitled complaints, I hope mine didn't come across as such! I believe it remains worthwhile to warn about insidious problems that users of the GA may be harmed by, even when they're not your fault at all, nor your responsibility to address. Not as a matter of liability, but of community benefit. I think re-opening the issue is a great idea. Although it's very noble you're willing to try solve the problem, it is (as you point out) not your responsibility and might be a poor use of time. I was content myself to roll-back the runner version to Caution A bug in MPICH on Of course the issue being open helps discoverability a lot - it's the first place many look when finding strange behaviour. In any case, thanks for all your brilliant efforts on this GA so far! |
mpi4py in a Github runner seems to have stopped being able to allocate multiple threads. We have tests that need exactly 4 threads to pass and mpi4py from
setup-mpi
seems to have stopped allocating anything other then 1, even tho it will still execute a given scriptn
times. For an example I am attaching a runner script, a sample mpi4py test, the incorrect, and the expected outputs.For some more background see our issue tracking this problem
Thanks in advance!
mpi_error.yml
test_mpi.py
output when github runner CI is used
The expected output (commands run on a local machine configured with mpi4py)
The text was updated successfully, but these errors were encountered: