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
Feature/pytorch backend #165
Feature/pytorch backend #165
Conversation
…ng methods separately did not work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great! Minor comments.
Thank you so much for doing this! Hopefully we can convince more researchers now to try to use TensorNetworks for their experiments. ^-^
@@ -160,7 +160,7 @@ additional-builtins= | |||
ignore-mixin-members=yes | |||
# List of classes names for which member attributes should not be checked | |||
# (useful for classes with attributes dynamically set). | |||
ignored-classes=SQLObject,twisted.internet.reactor,hashlib,google.appengine.api.memcache | |||
ignored-classes=SQLObject,twisted.internet.reactor,hashlib,google.appengine.api.memcache,torch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason you added this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is to fix this issue: pytorch/pytorch#701
I tried adding the separate functions that are used only, but for some reason that didn't work, so I reverted to the proposed solution of adding the full toch library. Tbh, I'm not 100% happy with this either.
…ature/pytorch-backend
This reverts commit 2e8bf33
First attempt to add PyTorch backend, expect a lot of comments & input on this PR and there is no hurry as I might not have time to address those quickly. Consider this a first check of progress.