Skip to content

Add logger configuration and support for command-line arguments#127

Merged
shima004 merged 3 commits intomainfrom
feat/logger-config
Mar 11, 2026
Merged

Add logger configuration and support for command-line arguments#127
shima004 merged 3 commits intomainfrom
feat/logger-config

Conversation

@shima004
Copy link
Member

This pull request introduces configurable logging to the ADF Python framework. It allows users to specify the logger's file name and log levels via both configuration files and command-line arguments, improving flexibility and observability. The default logger settings are now loaded from config/launcher.yaml or can be overridden at launch.

Logger configuration enhancements:

  • Added logger configuration options (file_name, file_level, stream_level) to config/launcher.yaml for centralized and customizable logging setup.
  • Introduced new configuration keys in ConfigKey for logger-related settings, enabling easy retrieval and management of logger configuration values.
  • Updated the configure_logger function to accept parameters for log file name and log levels, and to apply these settings to both file and stream handlers. [1] [2]

Launcher and CLI improvements:

  • Extended the launcher CLI to accept --logger-file-name, --logger-file-level, and --logger-stream-level arguments, allowing users to override logger settings at runtime.
  • Modified the launcher initialization to pass the appropriate logger configuration (from CLI arguments or config file) to configure_logger, ensuring logger setup reflects user preferences.

Project versioning:

  • Bumped the project version to 0.2.8 in pyproject.toml to reflect these enhancements.

@shima004 shima004 self-assigned this Mar 11, 2026
@shima004 shima004 force-pushed the feat/logger-config branch from 3a0c7d3 to 9dd2836 Compare March 11, 2026 13:49
@shima004 shima004 merged commit 01af085 into main Mar 11, 2026
4 checks passed
@shima004 shima004 deleted the feat/logger-config branch March 11, 2026 13:50
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