Replies: 5 comments
-
|
Thanks for working on this! Per our discussion on slack it sounds like the main change is to the default neighbor list?
Is this something that would be integrated into TorchSim CI? Curious what you have in mind. |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
-
Yes, that's probably the right place for it. I want to check both correctness and performance. |
Beta Was this translation helpful? Give feedback.
-
|
I'm going to continue testing on more-exotic cards, but the test suite passed on the AMD Radeon RX 7600. There was a warning about one MACE test, but that was already a float64 model being run as float32, so a 6% difference isn't entirely surprising.
|
Beta Was this translation helpful? Give feedback.
-
|
This should be moved to discussion along with the AMD neighborlist kernel request. I have had other people request this as well. The problem will always be acceleration of the models as they are most of the time the bottleneck. A intermediate solution is to have triton kernels. |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I'm working on a patch to support AMD consumer and datacenter GPUs (gfx1102/gfx942). I'm building a regression test suite and will be testing on a few CDNA3/4 and RDNA3/4 cards.
If anyone has specific suggestions about what to include, please comment here and I will incorporate them.
Beta Was this translation helpful? Give feedback.
All reactions