Skip to content

Comments

Tools: Simplify log parser, add unit tests, fix bugs in corner cases#845

Open
gardner48 wants to merge 14 commits intodevelopfrom
tools/remove-chainmap
Open

Tools: Simplify log parser, add unit tests, fix bugs in corner cases#845
gardner48 wants to merge 14 commits intodevelopfrom
tools/remove-chainmap

Conversation

@gardner48
Copy link
Member

  • Simplify log parsing to remove use of ChainMap
  • Use json.dumps to simplify printing log
  • Add unit tests for parsing log files
  • Fix bug in parsing lines where values contain =
  • Fix bug in regex for lines where keys or values contain brackets
  • Raise exception if updating a dictionary will overwrite an existing key
  • Convert values to number when parsing payload
  • Add better example of returned list in docs
  • Add simple usage examples to doc strings

@gardner48 gardner48 added this to the SUNDIALS Next milestone Feb 10, 2026
@gardner48 gardner48 changed the title Tools: Simplify log parser, fix bugs in corner cases Tools: Simplify log parser, add unit tests, fix bugs in corner cases Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant