Tidy some of the start up logging

hemelb-codes · May 20, 2024 · ebce493 · ebce493
1 parent 57f2017
commit ebce493
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/Code/geometry/GeometryReader.cc b/Code/geometry/GeometryReader.cc
@@ -88,10 +88,6 @@ namespace hemelb::geometry
             );
         }
 
-        log::Logger::Log<log::Info, log::Singleton>("Opened geometry file %s", dataFilePath.c_str());
-        // TODO: Why is there this fflush?
-        fflush(nullptr);
-
         log::Logger::Log<log::Debug, log::OnePerCore>("Reading file preamble");
         GmyReadResult geometry = ReadPreamble();
 
@@ -320,7 +316,7 @@ namespace hemelb::geometry
         constexpr auto MiB = std::size_t(1) << 20;
         constexpr std::size_t MAX_GMY_BUFFER_SIZE = 64U * MiB;
 
-        log::Logger::Log<log::Info, log::Singleton>("Buffering compressed site data onto owning processes.");
+        log::Logger::Log<log::Info, log::Singleton>("Streaming geometry data and caching required blocks.");
         log::Logger::Log<log::Debug, log::Singleton>("Maximum buffer size %lu B", MAX_GMY_BUFFER_SIZE);
 
         block_cache ans;

diff --git a/Code/main.cc b/Code/main.cc
@@ -23,10 +23,16 @@ int main(int argc, char *argv[])
     net::MpiCommunicator commWorld = net::MpiCommunicator::World();
 
     net::IOCommunicator hemelbCommunicator(commWorld);
-    try
-    {
+    log::Logger::Log<log::Info, log::Singleton>(
+      "Initialised MPI: %d nodes x %d processes per node = %d total",
+      hemelbCommunicator.GetLeadersComm().Size(),
+      hemelbCommunicator.GetNodeComm().Size(),
+      hemelbCommunicator.Size()
+    );
+
+    try {
       // Parse command line
-      configuration::CommandLine options = configuration::CommandLine(argc, argv);
+      auto options = configuration::CommandLine(argc, argv);
 
       // Start the debugger (if requested)
       debug::Init(options.GetDebug(), argv[0], commWorld);