-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Export multi-transform to ace-evaluator #22
Comments
@gabor1 your view would be useful here. |
I am worried about running lammps with julia on HPCs. It could be that it is the future, but it would be nice not to be totally dependent on this, at least in the short run. IT would also be much less work to reimplement the transforms in PACE. |
I don't think it is a quick hack to put the multi-transform into PACE. that's why I'm suggesting to think of both options. But we will take a look. |
@WillBaldwin0 - does your spline export take care of this? |
My splines export only does the pair potential at the moment, since I haven't been messing around with the full ACE for repulsion stuff, so no multi transform |
ok. but in principle it could be expanded to that, right? |
but we leave it open and discuss it at one of the next software meetings. |
Yes it could, I'm not sure how that works on the PACE end though |
I think the PACE end is already done that way. |
just following up on this, I think we are going splines on everything so looking into exporting that to PACE is now urgent. |
Not obvious to me where we should put our efforts:
The text was updated successfully, but these errors were encountered: