Updating benchmarks to add Rockpool JAX-accelerated benchmarks#185
Updating benchmarks to add Rockpool JAX-accelerated benchmarks#185DylanMuir wants to merge 2 commits intoopen-neuromorphic:mainfrom
Conversation
DylanMuir
commented
May 21, 2024
- Added several Jax and torch CUDA-graph benchmarks for Rockpool
- Updated Dockerfile to include some missed dependencies
- Added descriptive assertion error message to benchmarking function
* Updated Dockerfile to include some missed dependencies * Added descriptive assertion error message to benchmarking function
biphasic
left a comment
There was a problem hiding this comment.
Hey Dylan, thanks a lot for adding the Rockpool benchmarks. Is there someone in your team that can let them run on a GPU machine to verify that they work? Alternatively I can just merge them, but there is no guarantee that the next person will be able to run them.
|
Coming back to this, I'll try to run the benchmarks on an NVIDIA/Docker enabled machine. It's never trivial to get that working 🤣 |
|
@DylanMuir Heya Dylan, should we keep this PR opened? The site has changed a lot since this was proposed and I'd be happy to resolve the conflicts to get this merged. That said we haven't done a lot with these benchmarks and the python code really aught to be moved to its own repository if we do want to keep working on it. |