-
Notifications
You must be signed in to change notification settings - Fork 25.2k
🎉 Breaking changes 🎉 #455
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🎉 Breaking changes 🎉 #455
Conversation
@@ -3,203 +3,28 @@ | |||
import torch | |||
from .module import Module | |||
|
|||
def _addindent(s_, numSpaces): |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
self._modules = OrderedDict() | ||
# DeprecationWarning is ignored by default <sigh> | ||
warnings.warn("nn.Container is deprecated. All of it's functionality " | ||
"is not implemented in nn.Module. Subclass that instead.") |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
else: | ||
return CminConstant(other)(self) | ||
def reciprocal(self): | ||
return Cinv()(self) |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Update nn_tutorial to reflect UI change
…ytorch#455) Generated kernels are printed even with dynamic allocations. An error is thrown before compilation.
* Extract MKL version and build number into env vars * Use common/install_mkl.sh to install MKL * Rectify duplicate "Install MKL" step from libtorch/ubuntu16.04/Dockerfile
* Remove op overloads * lint
cmin
,cmax
andcinv
(functionality ofcmin
,cmax
split betweenmax
/min
andclamp
;cinv
renamed toreciprocal
)nn.Container
code intonn.Module
(nn.Container
will now start showing deprecation warnings)