Skip to content
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

General Radial Embedding #11

Open
cortner opened this issue Sep 23, 2023 · 1 comment
Open

General Radial Embedding #11

cortner opened this issue Sep 23, 2023 · 1 comment
Assignees

Comments

@cortner
Copy link
Member

cortner commented Sep 23, 2023

The equivariant_model is too rigid - as a user of the package / developer of models, I want to specify an arbitrary radial embedding. Here "radial" is a substitute for embedding everything that is invariant.

Specifically, I want to allow models that have Rnl and Ylm. The Rnl is just a function of xj i.e Rnl(xj) is some vector (embedding / basis). All the information how xj is transformed and then a basis is applied is hidden in there. The user/developer must guarantee that Rnl is invariant.

So the calling convention should be

radial, radial_spec = ... # construct the radial embedding
equivariant_model(AAspec, maxL, radial, radial_spec)

The radial basis is so application specific that it should not be automatically generated, i.e., no default. But we can provide a few simple radial basis sets.

The radial_spec could also be stored inside radial if we can give it meta information. It's just a list of (n, l) tuples, i.e.,

radial_spec = [ (n = 0, l = 0), (n = 1, l = 0), ... ] 

@zhanglw0521

@zhanglw0521
Copy link
Collaborator

zhanglw0521 commented Oct 19, 2023

Is this issue closed by #17 or is there anything missing? (I am guessing that this is resolved...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants