Pysyft: Missing docstrings in TorchHook
Some methods in TorchHook need docstrings. In particular, some of the original hooking methods (for example here and here and here) are missing them, as well as all the fixed precision ones (mainly all of the ones in this guy here).
jvmncs
All 6 comments
This is a great issue for people who want to understand more of TorchHook, since this will require clearly understanding and explaining the purpose of each of these methods and functions.
jvmncs
on 21 Jul 2018
👍4
I will take on this issue
kamathhrishi
on 27 Jul 2018
Are we still missing some docstrings?
alexandregranzerguay
on 7 Aug 2018
@kamathhrishi Any Update ?
I can check with you if you want ;)
jeamick
on 30 Aug 2018
If it's still open I will try to tackle this issue. First, I will try to create docstring for _hook_module.
g0lemXIV
on 6 Sep 2018
I am interested in taking a stab at the issue.
souravsingh
on 8 Oct 2018
Was this page helpful?
0 / 5 - 0 ratings
Related issues
wentaiwu92
路
4Comments
aristizabal95
路
3Comments
swaroopch
路
4Comments
mgale694
路
3Comments
deevashwer
路
4Comments
Most helpful comment
This is a great issue for people who want to understand more of TorchHook, since this will require clearly understanding and explaining the purpose of each of these methods and functions.