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

Benchmarking via pytest #316

Closed whitead closed 3 years ago

whitead commented 3 years ago

Work in progress. Aiming to incorporate bench marking into both CI and offline with GPU

pep8speaks commented 3 years ago

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

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2021-07-15 19:16:53 UTC
whitead commented 3 years ago

@geemi725 Ready for review. Added benchmarking code that tracks performance from commit to commit and generates html report that goes into documentation.

geemi725 commented 3 years ago

@whitead For some reason I couldn't generate the html report. When I did pytest -c pytest-benchmark.ini it gives the following error. 

ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --benchmark-autosave
  inifile: /gpfs/fs2/scratch/gwellawa/hoomd-tf/pytest-benchmark.ini
  rootdir: /gpfs/fs2/scratch/gwellawa/hoomd-tf

My current pytest version is 6.2.4

whitead commented 3 years ago

@geemi725 updated to help you install those packages

geemi725 commented 3 years ago

Works fine now!