Skip to content

Conversation

@lsawade
Copy link
Collaborator

@lsawade lsawade commented Nov 20, 2025

Description

Implements the logger struct and moves some things into the discussed namespaces.

  • Added the logger and the logger options structs (b14e64c)
  • Deprecated the context - context guard combination in favor of a single context struct (5d26ac2)
  • Forgot to add the header file (5b1316f)
  • Add new static struct specfem::MPI that holds rank, size, and comm information. (bfe6c98)
  • Renamed simulation to program and moved context namespace. (27e0214)

Issue Number

Updates #1149

Checklist

Please make sure to check developer documentation on specfem docs.

  • I ran the code through pre-commit to check style
  • THE DOCUMENTATION BUILDS WITHOUT WARNINGS/ERRORS
  • I have added labels to the PR (see right hand side of the PR page)
  • My code passes all the integration tests
  • I have added sufficient unittests to test my changes
  • I have added/updated documentation for the changes I am proposing
  • I have updated CMakeLists to ensure my code builds
  • My code builds across all platforms

icui and others added 30 commits October 22, 2025 14:36
1178 Part I - Remove write_adjacency_map flag
- [x] Adds a dimension tag to materials
- [x] Adds element tags within the mesher
- [x] Adds constructor to element types
- [x] Adds constructor to assembly properties
- [x] Adds new constructor for receivers
- [x] MESHFEM3D mesher now writes number of NGLL points
- [x] Added element grid to mesh struct
- [x] Added test to check element grid
Copy link
Collaborator

@Rohit-Kakodkar Rohit-Kakodkar left a comment

Choose a reason for hiding this comment

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

We need to talk about how to safely abort code in a hybrid Kokkos/MPI code.

Rohit-Kakodkar and others added 4 commits November 24, 2025 14:34
[Fix] Fixes meshfem2D for fluid solid coupling
1178 - Construct assembly::coupled_interfaces from adjacency_graph
gmshlayerbuilder: allow solids to skip AFS mark
@Rohit-Kakodkar Rohit-Kakodkar self-requested a review December 1, 2025 14:10
Copy link
Collaborator

@Rohit-Kakodkar Rohit-Kakodkar left a comment

Choose a reason for hiding this comment

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

Mostly LGTM

// Print/log error message if provided
if (!full_message.empty()) {
// Check if we're inside a valid program context
if (specfem::MPI_new::rank != -1 && specfem::MPI_new::size != -1) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be specfem::MPI_new::check_context()

}

// Check if we're inside a valid program context by checking MPI_new state
if (specfem::MPI_new::rank != -1 && specfem::MPI_new::size != -1) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same here.

…ork-depr-mpi-cout

Specfem Core Framework Depr Mpi Cout
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.

5 participants