llnl / llnl/sundials

Allow custom loggers

Open
#652 4 comments 0 reactions 0 assignees View on GitHub
feature-request
Dominant language
C
Stars
686
Forks
167
Avg merge
1d 20h
Merged PRs (30d)
17

Description

It looks like as part of the refactoring around logging/error handling, the option of being able to specify a function to control where logs go has disappeared. Currently, you can just set a filename to use, whereas previously you could tie it into other logging systems (e.g. Python's logging), or just drop the logs as needed (which you can sort-of do now by configuring the logfile as `/dev/null`). Would it be possible to have a logging version of https://sundials.readthedocs.io/en/latest/sundials/Errors_link.html#c.SUNErrHandlerFn (which appears not to be called for lower priority logs)?

Contributor guide

Open the contributing guide

Research direction

Start by reading the linked SUNErrHandlerFn documentation and locating the current logfile configuration and logging implementation. Determine where the refactoring removed callback-based logging and what existing tests cover; done means the issue's requested custom logging behavior is defined and verified without losing filename-based logging.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
hpc
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.