-
Notifications
You must be signed in to change notification settings - Fork 177
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
missing stuffs in torch.nn.utils #1243
Comments
Hi @yueyinqiu ! All contributions are more than welcome, so if you want to port some of the functions as well, that would be great! [Side note: The |
Well in fact I'm just a newbie to deep learning and I was trying reproducing some others' work which used Actually I'd like to contribute the project. It's so great to be able to use C# instead of Python. However I'm afraid that my unfamiliarity with deep learning and PyTorch will mess everything up :( (Please feel free to edit my list, like just remove the |
I've tried to add |
@yueyinqiu -- I just merged your PR. Please edit the to-do list in the first comment to reflect your changes. |
yes and thanks a lot |
@yueyinqiu -- I was taking a look at this again -- the fuse_***_eval methods, how commonly used are those? They seem straight-forward enough, but how important are they to have? |
Hmm... I'm not sure. I'm not really familiar with deep learning. But implementing this won't cost a lot, I think? Are you considering this because of #1259, and hesitating whether to have generalized types for |
|
I think that would be a good idea. |
rnn
is checked because the class do exist in TorchSharp, but actually not all the methods inside have been implemented.The text was updated successfully, but these errors were encountered: