FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[BBT#493] New API for parsec_init w/o cores parameter by abouteiller · Pull Request #314 · ICLDisco/parsec · GitHub

/ parsec Public

[BBT#493] New API for parsec_init w/o cores parameter - #314

Draft
abouteiller wants to merge 5 commits into
ICLDisco:masterfrom
abouteiller:cleanup/parsec_init
ConversationCommits (5)ChecksFiles changed
Draft

[BBT#493] New API for parsec_init w/o cores parameter#314
abouteiller wants to merge 5 commits into
ICLDisco:masterfrom
abouteiller:cleanup/parsec_init

Conversation

abouteiller commented Feb 24, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

https://bitbucket.org/icldistcomp/parsec/pull-requests/493

  • add info comm=%p to permit selection of comm before init
  • add parsec_info_create, use parsec_info_set to pass-in arguments to parsec before parsec_init
  • Dplasma corresponding request
  • rename setenv_mca_param_int to something less obnoxious
  • include definition of setenv_mca_param_int from public API
  • decide what to do with argc, argv
    • We will keep argc/argv so that command line parameters can be 'cleaned-up' and return only argc/argv that PaRSEC didn't consume, it should be valid to pass NULL, NULL.
  • update copyrights

Remove 'cores' parameter to parsec_init
2f5b155 
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
Cumulative upgrade to copyrights (and adding some missing copyrights
2ee5aa4 
based on git history).
Update the 'version' meta to parsec API 4.0
beac35c 
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
Rename parsec_setenv_mca_param_int to parsec_param_set_int
39aa36e 
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
Add the capability of changin the MPI comm before parsec_init
030221b 
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
abouteiller added the enhancement New feature or request label Feb 24, 2022
abouteiller added this to the v4.0 milestone Feb 24, 2022
abouteiller self-assigned this Feb 24, 2022

bosilca commented Oct 7, 2022

Copy link
Copy Markdown
Contributor

Discussion on 10/07/22: Add support for a simple API (based on the MPI Info) and we pass this as the last argument to parsec_init. We keep the pointers to argc and argv, but we do not change them. Look at issue #431 for more info.

abouteiller added the API Change Change to the public API, backward incompatible (version major bump) label Oct 12, 2022
abouteiller mentioned this pull request Oct 28, 2022
Merged

therault commented Dec 16, 2022

Copy link
Copy Markdown
Contributor

the info object should be a local that is passed explicitly to the set functions, and to the parsec_init call.

the developer should have a simple way of listing the info keys.

bosilca commented Mar 31, 2023

Copy link
Copy Markdown
Contributor

As discussed on 03/31/23 we need to merge this with #443 and get them in.

abouteiller modified the milestones: v4.0, v5.0 Jan 25, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

abouteiller

Labels

API Change Change to the public API, backward incompatible (version major bump) enhancement New feature or request

Projects

None yet

Milestone

v5.0

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL