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

Charges in LODE (Question) #127

Open
atulcthakur opened this issue Dec 14, 2024 · 1 comment
Open

Charges in LODE (Question) #127

atulcthakur opened this issue Dec 14, 2024 · 1 comment
Labels
question Further information is requested

Comments

@atulcthakur
Copy link

Hi,

Thanks for a great package. I have been reading the documentation and API to understand more about torch-pme. I am looking at the LODE example and it seems to be needing atomic charges as weights on the grids. As far as I understand, the original LODE does not depend on these atomic charges.
I wanted some clarification about why torch-pme needs atomic charges, when featomic can compute LODE without it (unless I am totally misunderstanding LODE construction)?
Also, what would be a recommended way for computing LODE? Featomic or torch-pme?

@ceriottm
Copy link
Contributor

As you can see, the LODE example is kind of a demo more than a full-fledged implementation. If you read the preprint accompanying the library, our impression is that LODE can be simplified a great deal without much accuracy loss.

If you need a full-fledged LODE implementation consistent with the original version, you better stick with featomic for the moment, but I'm planning to "promote" the example to some kind of rudimentary calculator, that should be much faster on GPUs, and for very large boxes (as well as much more flexible). In all event, if you want to replicate the per-element density of LODE, you need to simply use a one-hot encoding of the atom types (have multiple channels, and assign charges of one for one of the elements in each channel).

@PicoCentauri PicoCentauri added the question Further information is requested label Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants