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

90%

push

github

Compare sublayer names under their parents, filling a loop's index (#1264)

* Compare sublayer names under their parents, filling a loop's index

Decorating a layer's `call` drops its receiver's scope and those above
it, so its sublayers' variables collide only when two siblings under one
parent carry the same name. Names were compared across the whole tree,
and a name computed as `"h{}".format(i)` counted as colliding, so a stack
of blocks named from a loop's index was refused.

Names are now compared among siblings. A name `template.format(i)`, with
`i` the index of one `for` loop over `range`, is distinct per iteration.
A sublayer built in its parent's `__init__` or `build` is named apart
when its parents are, or carry unique names from Keras. The receiver
must be one object, since two receivers' sublayers collide.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NTnpjPjJj2aZy1ZkUD1eAn

* Refuse two templates under one parent that may fill a name alike

`"h{}"` over `range(11)` and `"h1{}"` over `range(1)` both fill `h10`,
but only equal templates were compared. A fill is the string of an
integer, so a shared name needs one template's text before its field to
start the other's and one's text after it to end the other's; templates
failing either, as `"h{}"` and `"ln_{}"` do, still release.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NTnpjPjJj2aZy1ZkUD1eAn

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

186 of 197 new or added lines in 1 file covered. (94.42%)

6776 of 7543 relevant lines covered (89.83%)

0.9 hits per line