Skip to content

docs: add MPICH#47

Open
hyoklee wants to merge 1 commit intoHDFGroup:masterfrom
hyoklee:patch-3
Open

docs: add MPICH#47
hyoklee wants to merge 1 commit intoHDFGroup:masterfrom
hyoklee:patch-3

Conversation

@hyoklee
Copy link
Member

@hyoklee hyoklee commented Feb 9, 2023

OpenMPI will fail test.
See #46 .

OpenMPI will fail test.
See HDFGroup#46 .
+ `libhdf5` - The [HDF5](https://www.hdfgroup.org/downloads/hdf5/) library.
Minimum version required is 1.13.3, compiled with
support for both parallel I/O and map objects
support for both [MPICH](https://www.mpich.org) parallel I/O and map objects
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We shouldn't mention a specific MPI implementation here since the connector will operate fine with implementations other than MPICH.

Copy link
Member Author

@hyoklee hyoklee Feb 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I attended DAOS tutorial today and @mchaarawi said MPICH is necessary.

@jhendersonHDF, are you going to patch CTest for openmpi? If not, I think user should be aware that MPICH is required to make test pass.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only use OpenMPI with the DAOS VOL connector, so no, MPICH isn't necessary. It's necessary currently for the tests to run via CTest, but it's not necessary in general. It's fine to document that MPICH is needed for the tests and/or to create an issue explaining that, but I don't think this is the correct place for that.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then, where is a good place to add this information for user?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally, we should just fix the issue so the tests aren't restricted to a particular MPI implementation.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should say HDF5 1.14.0 instead of 1.13.3. I don't want to make it sound like that's a legit, released version of the library.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably a misunderstanding. mpich is not necessary for the DAOS VOL.
mpich is necessary if you want to use the DAOS ADIO driver (MPIIO VFD - native VOL). there is a difference.

Copy link
Member Author

@hyoklee hyoklee Feb 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mchaarawi , thank you for the clarification.
I understood what you meant.
My point was: MPICH can work well for everything in DAOS stack so why should a DAOS user bother with OpenMPI?
Is there any extra benefit of using OpenMPI + HDF5 DAOS VOL?

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