-
Notifications
You must be signed in to change notification settings - Fork 24
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
PT-Gradient #127
PT-Gradient #127
Conversation
9e250b8
to
91cea54
Compare
* Simplify functionality state_gradient and compute_gradient_and_dynamics * Overhall and simplify tutorial * Overhall code style (pylint) * Correct several doc strings * Add numdifftools in requirements_ci * Several more minor changes
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.
Hi @eoin-dp-oneill.
Thank you for your contribution!
I've edited your code a bit in the last couple of days to make it ready for merging. I just pushed two commits to your branch implementing those changes.
One thing that I would need from you is to complete the two lines marked with '???' in gradients.py.
The compute_gradient_and_dynamics() function should a list of ndarrays (not tensornetwork.Node objects). However tensornetwork.Node.get_tensor() does not guarantee a specific order of axis when called without parameters. Could you (@eoin-dp-oneill) please specify the axes order and describe that order also in the docstring?
Thank you, and congratulations! |
PT-Gradient
Here is a pull-request for the PT-Gradient code, with modifications according to feedback from Gerald et al.
tox -e py36
(to runpytest
) the code tests.tox -e style
(to runpylint
) the code style tests.tox -e docs
(to runsphinx
) generate the documentation./docs/pages/modules.rst
has been updated./docs/pages/api.rst
has been updated.There will be a separate pull request for the Performance Testing code.
Hope everything is acceptable here. Let me know if any other improvements/fixes could be made.
Best,
Eoin O.