Lunderberg opened a new pull request, #15025:
URL: https://github.com/apache/tvm/pull/15025
Prior to this commit, use of `nn.Placeholder` or `nn.Parameter` outside of a
`with block_builder.function('name'):` scope resulted in an error. This commit
updates the behavior to allow declaration prior to entering the `with` block.
This can be useful for declaring a model object, which is then used to define
several related functions.
The scope was required so that `relax.BlockBuilder.current()` could
de-duplicate variable names. While two distinct variables in Relax may have
identical names, for user readability it is convenient to have all names be
unique within a Relax function. This commit maintains the de-duplication of
names if a `nn.Placeholder` or `nn.Parameter` is defined within an active
`relax.BlockBuilder`, that context may be used to provide a unique name.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
