-
Notifications
You must be signed in to change notification settings - Fork 1
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
Embeddings for (A)CE models #2
Comments
Does it mean that we need a more general |
Related to this - should we bring the States idea to this package? |
I can imagine it is useful to have the Also, it seems that |
Where did we leave the discussion about the |
We mentioned we can either have one more layer in between |
So specifically - did we have a firm decision on keeping the |
No, we didn't have a firm decision on this but just said that it is not the most urgent thing. And does it mean that this package will eventually serve as something like ACE3? I am somewhat for this idea, so will have totally no problem with it if we finally want |
I think it is time to discuss/iterate over this? I advocate for the |
We can try that. If I understand you right you want to take the
?? |
What in my mind is that we have a tiny package for If we decide to try this I think |
vote for this name too |
Ok, I’ll try to get something started tomorrow |
I am liking |
closed by creation of |
The initial prototype is a bit rigid in the class of particles systems. We need to import the very general ideas to construct embeddings from ACE.jl.
@zhanglw0521 you are a bit familiar with that, maybe you can give @CheukHinHoJerry an introduction? But don't get started on it yet.
The text was updated successfully, but these errors were encountered: