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

{data} [intel-2015a] fixes for mpi enabled h5py (review) #1825

Merged
merged 1 commit into from Aug 5, 2015

Conversation

JensTimmerman
Copy link
Contributor

this seems to have changed from 2.2 to 2.5

== 2015-08-05 10:21:15,214 main.run DEBUG cmd " python setup.py build  --mpi --hdf5=$EBROOTHDF5 " exited with exitcode 1 and output:
zip_safe flag not set; analyzing archive contents...

Installed /dev/shm/h5py/2.5.0/intel-2015a-Python-2.7.9-gpfs/h5py-2.5.0/.eggs/pkgconfig-1.1.0-py2.7.egg
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: option --mpi not recognized

@JensTimmerman JensTimmerman changed the title fixes for mpi enabled h5py {intel-2015a} [h5py] fixes for mpi enabled h5py (review) Aug 5, 2015
@JensTimmerman JensTimmerman changed the title {intel-2015a} [h5py] fixes for mpi enabled h5py (review) {data} [intel-2015a] fixes for mpi enabled h5py (review) Aug 5, 2015
@JensTimmerman
Copy link
Contributor Author

see also h5py/h5py#604

@boegel
Copy link
Member

boegel commented Aug 5, 2015

How did you notice this was missing? Any way to enhance the sanity check to catch this?

@hpcugentbot
Copy link
Contributor

Refer to this link for build results (access rights to CI server needed):
https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3863/
Easyconfigs unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-easyconfigs-pr-builder/3863/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

@JensTimmerman
Copy link
Contributor Author

running

from mpi4py import MPI
import h5py

rank = MPI.COMM_WORLD.rank  # The process ID (integer 0-3 for 4-process run)

f = h5py.File('parallel_test.hdf5', 'w', driver='mpio', comm=MPI.COMM_WORLD)

f.close()

when trying to use the mpio driver without having mpi enabled it raises errors that mpi4py is not imported inside h5py.

@boegel
Copy link
Member

boegel commented Aug 5, 2015

That's a bit elaborate as a sanity check... Couldn't find an easier way myself, so be it.

@boegel
Copy link
Member

boegel commented Aug 5, 2015

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
Linux SL 6.6, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.6.6
See https://gist.github.com/5de76b8aead54ff4a3c1 for a full test report.

boegel added a commit that referenced this pull request Aug 5, 2015
{data} [intel-2015a] fixes for mpi enabled h5py (review)
@boegel boegel merged commit 078b7ea into easybuilders:develop Aug 5, 2015
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.

None yet

3 participants