Skip to content

fix: remove accidental debug logging from decoder pipeline#4075

Open
harshaa765 wants to merge 1 commit into
AI-Hypercomputer:mainfrom
harshaa765:fix/remove-decoder-debug-logs
Open

fix: remove accidental debug logging from decoder pipeline#4075
harshaa765 wants to merge 1 commit into
AI-Hypercomputer:mainfrom
harshaa765:fix/remove-decoder-debug-logs

Conversation

@harshaa765
Copy link
Copy Markdown

Summary

  • Removes two max_logging.info(f"DEBUG: ...") statements that were accidentally committed to the pipeline scan path inside Decoder.__call__ (layers/decoders.py, previously lines 1029–1030).
  • These lines emitted on every decode step, polluting production inference logs with internal debug output.
  • No logic change — surrounding current_broadcast_args construction and scan_decoder_layers call are untouched.

Test plan

  • Python syntax verified locally (ast.parse passes).
  • Existing unit and integration tests for the decoder path cover the surrounding logic (no new test needed — this is a pure log-line deletion with no behavioral change).

Two max_logging.info(DEBUG: ...) statements were left in the pipeline
scan path inside Decoder.__call__, emitting on every decode step.
@google-cla
Copy link
Copy Markdown

google-cla Bot commented Jun 5, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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