ponder-lab / Hybridize-Functions-Refactoring / #1684

72%

main: 72%

Build Type

Pull #558

travis-pro

Committed by khatchad

Commit Message

Log a `WARNING` when an unknown `@tf.function` decorator argument is encountered

`Function.markParam` previously silently ignored decorator-argument
names it didn't recognize. The Javadoc on the switch justified the
choice as forward-compat with future TF versions, but silent-ignore
loses the signal—no test, evaluator output, or UI surface could see
that an unknown kwarg was used.

Add a `default` branch to the switch that logs the unknown argument
name (and the owning function) at `WARNING` level via the existing
`ILog LOG`. Forward-compat preserved (analysis still continues); the
signal is now surfaced in logs and CI output.

This is an intermediate step ahead of #204's original ask (custom
`UnknownHybridizationParameter` exception + matching test). The issue
stays open for the larger exception + test work.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Pull Request Pull Request #558: Log a `WARNING` when an unknown `@tf.function` decorator argument is encountered

Coverage Stats

0 of 1 new or added line in 1 file covered. (0.0%)

71 existing lines in 1 file now uncovered.

1188 of 1646 relevant lines covered (72.17%)

0.72 hits per line