samskalicky commented on a change in pull request #19386:
URL: https://github.com/apache/incubator-mxnet/pull/19386#discussion_r523240675
##########
File path: python/mxnet/gluon/block.py
##########
@@ -1149,30 +1156,43 @@ def hybridize(self, active=True, backend=None,
backend_opts=None, clear=True, **
The name of backend, as registered in `SubgraphBackendRegistry`,
default None
backend_opts : dict of user-specified options to pass to the backend
for partitioning, optional
Passed on to `PrePartition` and `PostPartition` functions of
`SubgraphProperty`
- clear : clears any previous optimizations
- static_alloc : bool, default False
+ clear : bool, default True
Review comment:
are we changing the clear default for optimize_for, but leaving the
default to True for hybridize? That seems to make sense, existing behavior for
hyridize was to clear on every call so that makes sense. Just checking
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
