Skip to content

Added logging print levels and verbosity#66

Merged
PatrickHenkel1 merged 5 commits into
mainfrom
65-add-verbose-setting
Feb 6, 2026
Merged

Added logging print levels and verbosity#66
PatrickHenkel1 merged 5 commits into
mainfrom
65-add-verbose-setting

Conversation

@PatrickHenkel1

@PatrickHenkel1 PatrickHenkel1 commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Description

Added logging print levels and verbosity

Type of Change

  • Bug fix (non-breaking change fixing an issue)
  • New feature (non-breaking change adding functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (code change that neither fixes a bug nor adds a feature)
  • Documentation update

Required Checklist

Testing

  • Unit tests have been created/updated for new/modified functionality
  • CI/CD pipeline passes all tests (pytest, coverage, executables, installation)

Examples

  • Add examples for new features and functionality

Compatibility

  • Changes are backward compatible OR deprecation warnings added
  • No breaking changes to public APIs
  • New dependencies added to pyproject.toml (required in dependencies or optional in [project.optional-dependencies])

Documentation

  • Docstrings added/updated for new/modified public methods (Google style)
  • Type hints added for new functions/methods

Breaking Changes

Optional

GitHub Copilot Review

  • Request Copilot review via GitHub UI (add 'copilot' as a reviewer)

Copilot AI review requested due to automatic review settings February 6, 2026 11:01
@PatrickHenkel1 PatrickHenkel1 linked an issue Feb 6, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 introduces a simple “print level” concept on the project Logger and wires it into model training/evaluation to control Keras verbosity and reduce unstructured print() usage across the codebase.

Changes:

  • Add Logger.print_level, Logger.print(), and Logger.verbosity() utilities and propagate them to Keras fit()/predict()/callbacks.
  • Replace various print() statements (and remove some model.summary() calls) with the new logger behavior.
  • Update an example executable to set print_level='warning'.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
physXAI/utils/logging.py Adds print levels + verbosity helpers and updates error/warning output to use Logger.print.
physXAI/preprocessing/constructed.py Attempts to route duplicate-registry warnings through Logger.print (currently via an incorrect import).
physXAI/models/models.py Uses Logger.verbosity() to control Keras predict verbosity and routes registry warnings through Logger.print.
physXAI/models/ann/model_construction/residual_models.py Removes unconditional model.summary() output.
physXAI/models/ann/model_construction/rbf_models.py Routes gamma debug output through Logger.print; removes unconditional model.summary().
physXAI/models/ann/model_construction/ann_models.py Removes unconditional model.summary() output.
physXAI/models/ann/ann_design.py Uses Logger.verbosity() for Keras fit + EarlyStopping verbosity; gates model.summary() behind print level.
physXAI/feature_selection/recursive_feature_elimination.py Replaces print() statements with Logger.print.
physXAI/evaluation/metrics.py Replaces metric print() calls with Logger.print and removes Metrics.print_evaluate.
executables/bestest_hydronic_heat_pump/P_hp.py Example sets logger print_level='warning'.

Comment thread physXAI/utils/logging.py Outdated
Comment thread physXAI/utils/logging.py Outdated
Comment thread physXAI/preprocessing/constructed.py Outdated
@PatrickHenkel1 PatrickHenkel1 merged commit 143dd97 into main Feb 6, 2026
13 checks passed
@PatrickHenkel1 PatrickHenkel1 deleted the 65-add-verbose-setting branch February 6, 2026 13:19
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.

Add verbose setting

3 participants