Skip to content

Added SST_MPI_Bcast, SST_MPI_Send, and SST_MPI_Recv to sst_mpi.h. Us…#1589

Merged
feldergast merged 2 commits intosstsimulator:develfrom
feldergast:output-mpi-updates
Apr 16, 2026
Merged

Added SST_MPI_Bcast, SST_MPI_Send, and SST_MPI_Recv to sst_mpi.h. Us…#1589
feldergast merged 2 commits intosstsimulator:develfrom
feldergast:output-mpi-updates

Conversation

@feldergast
Copy link
Copy Markdown
Contributor

…e these functions in perfReporter.cc to fix issues with compiling without MPI.

… these functions in perfReporter.cc to fix issues with compiling without MPI.
@feldergast feldergast requested a review from gvoskuilen April 16, 2026 14:51
@github-actions github-actions bot added AT: WIP Mark PR as a Work in Progress (No Autotesting Performed) AT: CLANG-FORMAT PASS and removed AT: WIP Mark PR as a Work in Progress (No Autotesting Performed) labels Apr 16, 2026
@github-actions
Copy link
Copy Markdown

CLANG-FORMAT TEST - PASSED

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request.

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements

  • Build Num: 2633
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MR-2

  • Build Num: 2578
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MT-2

  • Build Num: 2576
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-core_Make-Dist

  • Build Num: 1106
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_OSX-15-XC15-ARM2_OMPI-4.1.6_PY3.10_sst-elements

  • Build Num: 1121
  • Status: STARTED

Using Repos:

Repo: CORE (feldergast/sst-core)
  • Branch: output-mpi-updates
  • SHA: 930e2b8
  • Mode: TEST_REPO
Repo: SQE (sstsimulator/sst-sqe)
  • Branch: devel
  • SHA: f30c768973a466c3a0a22652d1eead26f647f836
  • Mode: SUPPORT_REPO
Repo: ELEMENTS (sstsimulator/sst-elements)
  • Branch: devel
  • SHA: d0a17fd573a1b5b23fa7d4348ab48da5bcd48869
  • Mode: SUPPORT_REPO
Repo: MACRO (sstsimulator/sst-macro)
  • Branch: devel
  • SHA: 04e5625dc306d5a1b29535c2d2a0c4b8b3cb9cce
  • Mode: SUPPORT_REPO

Pull Request Author: feldergast

gvoskuilen
gvoskuilen previously approved these changes Apr 16, 2026
Copy link
Copy Markdown
Contributor

@gvoskuilen gvoskuilen left a comment

Choose a reason for hiding this comment

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

Looks good other than a minor typo.

Comment thread src/sst/core/sst_mpi.h Outdated
// MPI_Barrier is a no-op when MPI is not enabled and can be safely called in that case
int SST_MPI_Barrier(MPI_Comm comm);

// MPI_Send and MPI_Recv are both no-ops if NPI is not present. However, they should be in segments of code that won't
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"NPI" -> "MPI"

@github-actions github-actions bot added AT: WIP Mark PR as a Work in Progress (No Autotesting Performed) and removed AT: WIP Mark PR as a Work in Progress (No Autotesting Performed) labels Apr 16, 2026
@github-actions
Copy link
Copy Markdown

CLANG-FORMAT TEST - PASSED

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - Error: Jenkins Jobs - A user has pushed a change to the PR before testing completed. NEW EVENT 'committed', ID C_kwDOApdhgNoAKGU1YWE5ZGE5NzViNWQwNjMwZjk4ZDIwYzY0NmRlMGZjOTU3MDQ4NzA... The Jenkins Jobs will be shutdown; Testing of this PR must occur again.

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - Jenkins Testing: 1 or more Jobs FAILED

Note: Testing will normally be attempted again in approx. 4 Hrs. If a change to the PR source branch occurs, the testing will be attempted again on next available autotester run.

Pull Request Auto Testing has FAILED (click to expand)

Job: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements

  • Result: PASSED
  • Build #: 2633
  • URL: Jenkins server at https://sst-jenkins.sandia.gov/view/SST/job/SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements/2633/consoleFull

Job: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MR-2

  • Result: FAILED
  • Build #: 2578
  • URL: Jenkins server at https://sst-jenkins.sandia.gov/view/SST/job/SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MR-2/2578/consoleFull
  • Job: - Status: None

Job: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MT-2

  • Result: FAILED
  • Build #: 2576
  • URL: Jenkins server at https://sst-jenkins.sandia.gov/view/SST/job/SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MT-2/2576/consoleFull
  • Job: - Status: None

Job: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-core_Make-Dist

  • Result: FAILED
  • Build #: 1106
  • URL: Jenkins server at https://sst-jenkins.sandia.gov/view/SST/job/SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-core_Make-Dist/1106/consoleFull
  • Job: - Status: FAILURE

Test Results

Test Name Status
test_RealTime_SIGUSR1_heartbeat FAILED
test_python_parallel_load_setnonlocal SKIPPED
test_integration_nightly SKIPPED
test_integration_pr SKIPPED
test_integration_weekly SKIPPED

Job: SST__AutotestGen2_NewFW_OSX-15-XC15-ARM2_OMPI-4.1.6_PY3.10_sst-elements

  • Result: PASSED
  • Build #: 1121
  • URL: Jenkins server at https://sst-jenkins.sandia.gov/view/SST/job/SST__AutotestGen2_NewFW_OSX-15-XC15-ARM2_OMPI-4.1.6_PY3.10_sst-elements/1121/consoleFull

@gvoskuilen gvoskuilen added the AT: RETEST Force PR to be tested (if inspections allow it) label Apr 16, 2026
@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - User Requested Retest - Label AT: RETEST will be reset after testing.

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection is Not Necessary for this Pull Request.

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements

  • Build Num: 2634
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MR-2

  • Build Num: 2579
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MT-2

  • Build Num: 2577
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-core_Make-Dist

  • Build Num: 1107
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_NewFW_OSX-15-XC15-ARM2_OMPI-4.1.6_PY3.10_sst-elements

  • Build Num: 1122
  • Status: STARTED

Using Repos:

Repo: CORE (feldergast/sst-core)
  • Branch: output-mpi-updates
  • SHA: e5aa9da
  • Mode: TEST_REPO
Repo: SQE (sstsimulator/sst-sqe)
  • Branch: devel
  • SHA: f30c768973a466c3a0a22652d1eead26f647f836
  • Mode: SUPPORT_REPO
Repo: ELEMENTS (sstsimulator/sst-elements)
  • Branch: devel
  • SHA: d0a17fd573a1b5b23fa7d4348ab48da5bcd48869
  • Mode: SUPPORT_REPO
Repo: MACRO (sstsimulator/sst-macro)
  • Branch: devel
  • SHA: 04e5625dc306d5a1b29535c2d2a0c4b8b3cb9cce
  • Mode: SUPPORT_REPO

Pull Request Author: feldergast

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements

  • Build Num: 2634
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MR-2

  • Build Num: 2579
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-elements_MT-2

  • Build Num: 2577
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_NewFW_sst-test_OMPI-4.1.4_PY3.9_sst-core_Make-Dist

  • Build Num: 1107
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_NewFW_OSX-15-XC15-ARM2_OMPI-4.1.6_PY3.10_sst-elements

  • Build Num: 1122
  • Status: PASSED

@sst-autotester sst-autotester removed the AT: RETEST Force PR to be tested (if inspections allow it) label Apr 16, 2026
@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ gvoskuilen ]!

@sst-autotester
Copy link
Copy Markdown
Contributor

Status Flag 'Pull Request AutoTester' - Pull Request MUST BE MERGED MANUALLY BY Project Team - This Repo does not support Automerge

@feldergast feldergast merged commit 41d67db into sstsimulator:devel Apr 16, 2026
10 checks passed
@feldergast feldergast deleted the output-mpi-updates branch April 16, 2026 16:21
kpgriesser added a commit to tactcomplabs/sst-core that referenced this pull request Apr 17, 2026
* N2m restart (sstsimulator#1586)

* Initial implementation of repartitioned restarts.

* Fix realtime checkpoint test to filter directory listing on directories before looking for checkpoint files.

* Add default constructor to SimpleNetwork::NetworkInspector to enable checkpointing for derived classes.

* Centralize profiling output from multiple sources (sstsimulator#1587)

* Centralize profiling output from multiple sources
Enable JSON format output for all profiling data

* Update tests to reflect new options and output format
Change `--print-timing-info` to just `--timing-info`

* Few minor format updates

* Added SST_MPI_Bcast, SST_MPI_Send, and SST_MPI_Recv to sst_mpi.h.  Us… (sstsimulator#1589)

* Added SST_MPI_Bcast, SST_MPI_Send, and SST_MPI_Recv to sst_mpi.h.  Use these functions in perfReporter.cc to fix issues with compiling without MPI.

---------

Co-authored-by: Scott Hemmert <[email protected]>
Co-authored-by: gvoskuilen <[email protected]>
Co-authored-by: Kenneth Griesser <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants