Skip to content

Better toml parm parse#26

Merged
baperry2 merged 10 commits intodevelopmentfrom
better-TomlParmParse
Dec 16, 2025
Merged

Better toml parm parse#26
baperry2 merged 10 commits intodevelopmentfrom
better-TomlParmParse

Conversation

@baperry2
Copy link
Collaborator

Use TOML files for config, but with auto-documentation and auto-save features.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modernizes the parameter parsing infrastructure by replacing the simple toml library with tomlkit and significantly enhancing the TomlParmParse class with auto-documentation, auto-save features, and improved command-line integration.

Key Changes:

  • Complete rewrite of TomlParmParse class with new features including nested parameter access via dict-like interface, automatic documentation generation, command-line argument parsing, and unused input detection
  • Migration from toml library to tomlkit for better control over formatting and comments
  • Updated example scripts to use the new parse_args() API with inline documentation strings

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 13 comments.

Show a summary per file
File Description
cmlm/utils/input_file.py Complete rewrite with new TomlParmParse implementation featuring auto-documentation, command-line parsing, and nested parameter access
run_scripts/ctable/create_spray_table_nd.py Updated to use new TomlParmParse.parse_args() API with doc strings for each parameter
run_scripts/ctable/create_spray_table_nd.toml Reformatted with enhanced comments and documentation structure
run_scripts/ctable/create_dummy_table_nd.py Updated to use new TomlParmParse.parse_args() API
run_scripts/ctable/create_dummy_table_nd.toml Reformatted with enhanced comments
pyproject.toml Changed dependency from toml >= 0.10.0 to tomlkit >= 0.13.0
cmlm/utils/__init__.py Minor whitespace formatting change

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@baperry2 baperry2 merged commit 5a8d6bb into development Dec 16, 2025
7 checks passed
@baperry2 baperry2 deleted the better-TomlParmParse branch December 16, 2025 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant