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

Multiple commits #1785

Merged
merged 7 commits into from
Sep 4, 2023
Merged

Multiple commits #1785

merged 7 commits into from
Sep 4, 2023

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Sep 4, 2023

docs: fix HTML word wapping in table cells

The sphinx_rtd_theme does not properly handle wrapping long lines in
table cells when rendering to HTML due to a CSS issue (see
readthedocs/sphinx_rtd_theme#1505).

Until the issue is fixed upstream in sphinx_rtd_theme, we can simply
override the CSS here (in conf.py).

The PRRTE docs don't use the RST ".. list-table::" directive much, so
this change won't really have much of an effect here. However, OMPI
and PMIx were updated with this conf.py change, so we might as well
keep all 3 projects more-or-less in sync.

Signed-off-by: Jeff Squyres jeff@squyres.com
(cherry picked from commit 9ef7478)

Support fix for OMPI Github issue #11532

Refactor parameter translation routine into a separate routine to make it
easier to copy into OMPI and maintain concurrently in both packages.

Signed-off-by: Quincey Koziol qkoziol@amazon.com
(cherry picked from commit d8c4c9e)

Fix binding to multiple pe's

Accept a package if it has enough pe's to satisfy
the binding - do not require that it have more
than required.

Thanks to Jakub Benda for the report and the fix!

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit d659897)

Fix typos in the --enable-dlopen support

Confusion over calling it prtedlopen vs dlopen.
Make it track the PMIx equivalent

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit 86b4ec6)

Fix typo of variable name (PRRTE->PRTE)

Thanks to Jeff Squyres for pointing it out.

Signed-off-by: Ralph Castain rhc@pmix.org
(cherry picked from commit f4c9fdd)

Remove Doxygen

We haven't used Doxygen in years; remove all vestigates it from the
repo.

Signed-off-by: Jeff Squyres jeff@squyres.com
(cherry picked from commit 32f023b)

bot:notacherrypick

jsquyres and others added 7 commits September 4, 2023 07:27
The sphinx_rtd_theme does not properly handle wrapping long lines in
table cells when rendering to HTML due to a CSS issue (see
readthedocs/sphinx_rtd_theme#1505).

Until the issue is fixed upstream in sphinx_rtd_theme, we can simply
override the CSS here (in conf.py).

The PRRTE docs don't use the RST ".. list-table::" directive much, so
this change won't really have much of an effect here.  However, OMPI
and PMIx were updated with this conf.py change, so we might as well
keep all 3 projects more-or-less in sync.

Signed-off-by: Jeff Squyres <jeff@squyres.com>
(cherry picked from commit 9ef7478)
Refactor parameter translation routine into a separate routine to make it
easier to copy into OMPI and maintain concurrently in both packages.

Signed-off-by: Quincey Koziol <qkoziol@amazon.com>
(cherry picked from commit d8c4c9e)
Accept a package if it has enough pe's to satisfy
the binding - do not require that it have _more_
than required.

Thanks to Jakub Benda for the report and the fix!

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit d659897)
Confusion over calling it prtedlopen vs dlopen.
Make it track the PMIx equivalent

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit 86b4ec6)
Thanks to Jeff Squyres for pointing it out.

Signed-off-by: Ralph Castain <rhc@pmix.org>
(cherry picked from commit f4c9fdd)
We haven't used Doxygen in years; remove all vestigates it from the
repo.

Signed-off-by: Jeff Squyres <jeff@squyres.com>
(cherry picked from commit 32f023b)
Signed-off-by: Ralph Castain <rhc@pmix.org>
@openpmix openpmix deleted a comment from github-actions bot Sep 4, 2023
@rhc54 rhc54 merged commit f0f7681 into openpmix:v3.1 Sep 4, 2023
4 of 5 checks passed
@rhc54 rhc54 deleted the cmr31/up branch September 4, 2023 16:32
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