refactor: Centralize error output #3902

arng40 · 2025-11-05T09:53:01Z

Remove code duplication found in GEOS_THROW, GEOS_ERROR, GEOS_WARNING and put into a static function in ErrorLogger.
Called while flushErrorMsg().
Move all Exceptions under GeosExceptions.hpp

…file

…necessary

… link between GEOS_THROW_CTX_IF and LVARRAY_THROW_IF_TEST( EXP, MSG, TYPE )

… in try/catch statements Problem: Retrieves everything that was thrown, so not just the message.

…/catch in main)": remove useless try/catch

…y spaces. The previous condition checked whether an argument was present and whether the option was immediately followed by a value like -test"value", which excluded valid cases like -test "value" et -test "value".

…t is expected.

src/coreComponents/common/logger/ErrorHandling.hpp

src/coreComponents/common/logger/ErrorHandling.cpp

src/coreComponents/common/logger/ErrorHandling.hpp

src/coreComponents/common/logger/ErrorHandling.cpp

src/coreComponents/common/logger/ErrorHandling.hpp

src/main/main.cpp

…larity

src/coreComponents/common/logger/ErrorHandling.cpp

src/coreComponents/common/logger/ErrorHandling.hpp

src/coreComponents/dataRepository/unitTests/testErrorHandling.cpp

src/coreComponents/common/logger/ErrorHandling.hpp

MelReyCG · 2025-12-29T16:35:14Z

src/coreComponents/common/initializeEnvironment.cpp

+      ErrorLogger::global().beginLogger()
+        .addSignalToMsg( signal )
+        .setType( ErrorLogger::MsgType::Error )
+        .addRank( ::geos::logger::internal::g_rank )
+        .addCallStackInfo( stackHistory )
+        .addContextInfo(
+        ErrorContext{ { { ErrorContext::Attribute::Signal, std::to_string( signal ) } }, 1 },
+        ErrorContext{ { { ErrorContext::Attribute::DetectionLoc, string( "signal handler" ) } }, 0 } )
+        .flush();


Maybe a unit test for signals would be a good thing.

src/coreComponents/common/logger/ErrorHandling.hpp

MelReyCG · 2026-01-08T16:04:18Z

src/coreComponents/dataRepository/unitTests/testErrorHandling.cpp

+      "***** - {}\n",
+      testErrorLogger.getCurrentExceptionMsg().m_file, line1,
+      testErrorLogger.getCurrentExceptionMsg().m_cause,
+      context.toString(),


Shouldn't you switch with the most important context here?

I still have the same remaks

src/coreComponents/common/logger/GeosExceptions.hpp

src/coreComponents/dataRepository/unitTests/testErrorHandling.cpp

src/coreComponents/common/logger/ErrorHandling.hpp

MelReyCG · 2026-01-08T16:36:55Z

src/coreComponents/common/logger/ErrorHandling.hpp

   * @note - local instances are possible for more specialized logging.
   *       - currently not available on GPU, use GEOS_WARNING/ERROR/ASSERT macros for this usecase.
   */
  GEOS_HOST static ErrorLogger & global();


I see your point, but just use thread_local here

Also I was wondering, what about the thread_local instances initialization?

src/coreComponents/common/logger/ErrorHandling.hpp

MelReyCG · 2026-01-08T16:54:09Z

src/coreComponents/common/logger/ErrorHandling.hpp

+  void flushCurrentExceptionMessage();
+
+  /**
+   * @brief Write all the information retrieved about the diagnostic message into the instance
+   * outputs (stream specified + optional yaml file)
+   * @param errorMsg a reference to the ErrorMsg to output, and will be re-initialized
+   * @note Used for warnings and non-exception errors
+   */
+  void flushErrorMsg( DiagnosticMsg & errMsg );


provide a oss parameter (with a default to std::cout)?

Sorry, my advise seems wrong as std::ostream & m_stream = std::cout; exists.

src/coreComponents/common/logger/ErrorHandling.hpp

MelReyCG · 2026-01-12T09:31:16Z

src/coreComponents/common/logger/GeosExceptions.hpp

+    m_formattingOSS.str("");  
+    m_formattingOSS.clear();  

+    ErrorLogger::writeToAscii( msg, m_formattingOSS );  
+    m_cachedWhat = m_formattingOSS.bad() ? "Exception formatting error!" : m_formattingOSS.str();  
  }

 private:
  /// Formatted exception message for what() method
  string m_cachedWhat;
-  /// Error message logger for structured error reporting
-  DiagnosticMsg m_errorMsg;
+  inline static thread_local std::ostringstream m_formattingOSS; 


MelReyCG · 2026-01-12T09:43:12Z

src/coreComponents/common/logger/Logger.hpp

-        .addCallStackInfo( LvArray::system::stackTrace( true ) ) \
-        .addToMsg( __msgoss.str() ) \
-        .addContextInfo( GEOS_DETAIL_REST_ARGS( __VA_ARGS__ )).get(); \
+      DiagnosticMsg exceptionMsg =  GEOS_GLOBAL_LOGGER.initCurrentExceptionMessage() \


wouldn't initCurrentExceptionMessage() need

std::string_view msgContent, integer rank, std::string_view msgFile, integer msgLine

?

MelReyCG · 2026-01-12T09:47:05Z

src/main/main.cpp

-                                           .addToMsg( e.what() )
-                                           .addRank( ::geos::logger::internal::g_rank )
+    DiagnosticMsg diagnosticMsg;
+    ErrorLogger::global().flushErrorMsg( DiagnosticMsgBuilder::init( diagnosticMsg,


not initCurrentExceptionMessage()?

MelReyCG · 2026-01-12T09:49:39Z

src/coreComponents/common/logger/Logger.hpp

        .addToMsg( __msgoss.str() ) \
-        .addContextInfo( GEOS_DETAIL_REST_ARGS( __VA_ARGS__ )).get(); \
+        .addContextInfo( GEOS_DETAIL_REST_ARGS( __VA_ARGS__ )) \
+        .getDiagnosticMsg(); \


There are a lot of unnecessary getDiagnosticMsg() in all usages.

MelReyCG · 2026-01-12T09:56:46Z

src/coreComponents/common/initializeEnvironment.cpp

+                                             ErrorContext{"",
+                                                          { { ErrorContext::Attribute::Signal,
+                                                            std::to_string( signal ) } },
+                                                          1 },


we could do that automatically in addSignalToMsg(), what do you think? (it would then be named addSignal())

amandinehry added 30 commits April 25, 2025 17:05

creation of an object header that logs errors and warnings in a yaml …

c846586

…file

Set up ErrorHandling

f1713fa

traits errors commented (to treat)

4e5250f

Set up test

94b775c

Modification to properly format the yaml so that it can be parsed if …

1cb8aa3

…necessary

Test to add variadic arguments

9d5e16a

draft: retrieve data contexts info

2f7660c

Added additional information in the yaml - not functional missing the…

6036551

… link between GEOS_THROW_CTX_IF and LVARRAY_THROW_IF_TEST( EXP, MSG, TYPE )

First attempt at outputting exceptions

46b4904

Complete: code for outputting errors into yaml

e1aed93

Manage the text contained in the potential exception

bc953be

First attempt: Handling the text contained in the potential exception…

1a434bc

… in try/catch statements Problem: Retrieves everything that was thrown, so not just the message.

Complete: yaml output for PVT tables error

acf4d99

YAML error message well formatted

ee58884

Fix duplication error

32a0237

Add variadic parameters to GEOS_THROW_CTX_IF

589270f

Reorganization after operational tests

9b65581

Apply the output error functionality in yaml to all geos errors

b13f0a6

Latest update for exception handling (try/catch in main)

6de2c6a

Add priority to the error message into the yaml file

5830aee

Update of the commit named "latest update for exception handling (try…

2b27c41

…/catch in main)": remove useless try/catch

set priority

86f793e

errors cli modification: add --errors-output

8a8117e

errors cli modification: add --errors-output

2c0f5aa

Replace original macros

eb2f691

Correction: replace original macros

7b5f293

Reacting with the command line option -e / --errors-output

670ddfa

Add the rank on which the error is catch for error and warning outputs

d08a30f

Unit test to check if the contents of the yaml file match exactly wha…

5a6c1a0

…t is expected.

arng40 and others added 3 commits December 10, 2025 17:57

Uncrustify

ab9895b

Merge branch 'develop' into refactor/dudes/error-context

60a1f3a

Merge branch 'develop' into refactor/dudes/error-context

4e74b00

MelReyCG requested changes Dec 17, 2025

View reviewed changes

arng40 and others added 6 commits December 19, 2025 15:42

reorder function + add constructor to ErrorContext

08973a3

uncrustify & doxygen

cee0059

Merge branch 'develop' into refactor/dudes/error-context

2a639a7

Merge branch 'develop' into refactor/dudes/error-context

a2b31d1

improve writeToAscii

d4db78b

add context to std::exception

7df78ff

arng40 mentioned this pull request Dec 23, 2025

feat: Error reporting at the end of the simulatiuon #3929

Draft

arng40 added 4 commits December 24, 2025 15:33

fix encapsulation in ErrorLogger by adding BuilderPattern + improve c…

abc6d2a

…larity

uncrustify

ec7ddc1

remove redundant lmsg in case of multiple rank

4a5b4fe

missing const

35d8707

MelReyCG requested changes Dec 29, 2025

View reviewed changes

MelReyCG reviewed Jan 5, 2026

View reviewed changes

src/coreComponents/common/logger/ErrorHandling.hpp Outdated Show resolved Hide resolved

src/coreComponents/common/logger/ErrorHandling.hpp Outdated Show resolved Hide resolved

arng40 added 5 commits January 5, 2026 14:22

delete ErrorMsg method & remove unecessary method in Builder

38023c2

extract inerclass & refacto ErrorBuilder

62fcf31

rename ErrorMsg & adapt errorLogger function called in catch

1495a68

modify yaml & renaming & fix

a4e69cd

remove debug log

16057c1

MelReyCG requested changes Jan 8, 2026

View reviewed changes

arng40 added 5 commits January 9, 2026 14:35

update prepareWhat

3ed52be

update builder

e40167f

rename get method & sort errorContext

b529d7a

remove debug

ec4703c

update format

fbed3ed

MelReyCG reviewed Jan 12, 2026

View reviewed changes

refactor: Centralize error output #3902

Are you sure you want to change the base?

refactor: Centralize error output #3902

Uh oh!

Conversation

arng40 commented Nov 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MelReyCG Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

arng40 commented Nov 5, 2025 •

edited

Loading

MelReyCG Jan 12, 2026 •

edited

Loading