pytorch
438cc79f - Improve more the error message with explicit recommendation

Commit
2 years ago
Improve more the error message with explicit recommendation Following feedback. The new message looks like: ``` # torch.nn.intrinsic.modules._FusedModule: - Is public: it is inside the module's (`torch.nn.intrinsic.modules`) `__all__` - Does NOT look public: because it starts with `_` (`_FusedModule`) - You can do either of these two things to fix this problem: - To make it NOT public: remove it from the modules's (`torch.nn.intrinsic.modules`) `__all__` - To make it look public: remove the `_` at the beginning of the name # torch.ao.nn.sparse.quantized.dynamic.linear.LinearBlockSparsePattern: - Is public: it is an attribute that does not start with `_` on a module that does not have `__all__` defined - Does NOT look public: because its `__module__` attribute (`torch.ao.nn.sparse.quantized.utils`) is not within the torch library or does not start with the submodule where it is defined (`torch.ao.nn.sparse.quantized.dynamic.linear`) - You can do either of these two things to fix this problem: - To make it NOT public: either define a `__all__` for `torch.ao.nn.sparse.quantized.dynamic.linear` or add a `_` at the beginning of the name - To make it look public: make sure the `__module__` is properly set and points to a submodule of `torch.ao.nn.sparse.quantized.dynamic.linear` ``` Pull Request resolved: https://github.com/pytorch/pytorch/pull/76261 Approved by: https://github.com/NivekT
Author
Committer
Parents
Loading