search

ur-whitelab / hoomd-tf

A plugin that allows the use of Tensorflow in Hoomd-Blue for GPU-accelerated ML+MD
https://hoomd-tf.readthedocs.io
MIT License
30 stars 8 forks source link

Update docs #289

Closed mehradans92 closed 3 years ago

pep8speaks commented 3 years ago

Hello @mehradans92! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 339:74: W291 trailing whitespace

Comment last updated at 2021-02-26 20:07:19 UTC
mehradans92 commented 3 years ago

@geemi725 Could you please add a little bit of documentation here? https://github.com/ur-whitelab/hoomd-tf/blob/c67c1c69c2cdfd40163a460204a01dd535b40937/htf/utils.py#L329

geemi725 commented 3 years ago

@mehradans92 I addressed the Pep8 issues and updated the docs for some functions I missed yesterday. This is ready for merging!

mehradans92 commented 3 years ago

@geemi725 Thanks for your review. htf.center_of_mass was also missing some details. Modified doc string.