Clean-Up - Githubissues

NWChemEx / ParallelZone

You're travelling through another dimension, a dimension not only of CPUs and threads but of GPUs; a journey into a wondrous land whose boundaries are bandwidth limited. That's the signpost up ahead - your next stop, the ParallelZone!

https://nwchemex.github.io/ParallelZone/

Apache License 2.0

0 stars 1 forks source link

Clean-Up #43

Closed ryanmrichard closed 1 year ago

ryanmrichard commented 2 years ago

In light of the current cacheing/memoization strategy there's some clean-up to be done in PZ.

remove BPHash and the associated hashing infrastructure
- If someone has a good reason to keep it speak now or forever hold your peace.
Much of the current serialization infrastructure can be dropped too
We probably want to use ArchiveWrapper, or something similar, to insulate downstream dependencies from the serialization details

jwaldrop107 commented 2 years ago

As an added detail around the removal of BPHash, we will want to replace at least some the hashing done with BPHash with std::hash specializations. Those additions are most likely going to occur in other repos (TensorWrapper, Chemist, and PluginPlay primarily) that are currently dependent on BPHash through this repo.