✨ Support non-instance value: class and post-hook #18
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR enhance the config rule, introduce the following features:
$
prefix to mark the input isclass
itself instead of its instance.<class BasicBlock>
instead of instance of BasicBlockoutpout of method parameters() of FCN
, exempting to define confighook.@
in the above optimizer example is necessary.FCN.net.block.parameters
Some defects:
@
and.method
is cooperating well, more situation should be tested$@parameters = "Model.build.parameters"
. So the only way to solve this situation is to use config hook or define a function witch return the output ofModel.build
in such a case.