Skip to content

Route Z-Image ControlNet diagnostics through logging#9289

Open
wunianze666-netizen wants to merge 1 commit into
invoke-ai:mainfrom
wunianze666-netizen:fix/z-image-controlnet-logging
Open

Route Z-Image ControlNet diagnostics through logging#9289
wunianze666-netizen wants to merge 1 commit into
invoke-ai:mainfrom
wunianze666-netizen:fix/z-image-controlnet-logging

Conversation

@wunianze666-netizen

Copy link
Copy Markdown

Summary

  • replace unconditional [DEBUG] / [WARNING] terminal output in z_image_controlnet_extension.py with module logger calls
  • add a regression test that checks initialization diagnostics do not write to stdout and still emit logging calls

Why

These diagnostics come from backend/library code and currently print directly to stdout, which can pollute terminal output and make log handling inconsistent for downstream callers.

Validation

  • added a regression test for constructor diagnostics behavior
  • verified the updated source and test files with py_compile
  • ran a source-level regression check to confirm the debug output is no longer implemented with direct print(...) calls

Note

The full pytest dependency set for this repository was not available in the current local environment, so I validated this change with targeted static checks plus compile-time verification.

@github-actions github-actions Bot added python PRs that change python files backend PRs that change backend files python-tests PRs that change python tests labels Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend PRs that change backend files python PRs that change python files python-tests PRs that change python tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant