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

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

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

abouteiller
Copy link
Contributor

@abouteiller abouteiller commented Feb 24, 2022

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

Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
Signed-off-by: Aurelien Bouteiller <bouteill@icl.utk.edu>
@abouteiller abouteiller added the enhancement New feature or request label Feb 24, 2022
@abouteiller abouteiller added this to the v4.0 milestone Feb 24, 2022
@abouteiller abouteiller self-assigned this Feb 24, 2022
@bosilca
Copy link
Contributor

bosilca commented Oct 7, 2022

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 abouteiller added the API Change Change to the public API, backward incompatible (version major bump) label Oct 12, 2022
@therault
Copy link
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
Copy link
Contributor

bosilca commented Mar 31, 2023

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

@abouteiller abouteiller modified the milestones: v4.0, v5.0 Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Change Change to the public API, backward incompatible (version major bump) enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants