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

supporting AtomsBase #120

Open
rkurchin opened this issue Mar 25, 2022 · 3 comments
Open

supporting AtomsBase #120

rkurchin opened this issue Mar 25, 2022 · 3 comments

Comments

@rkurchin
Copy link

👋 Hi there!

As you may be aware, some of us over at JuliaMolSim have been developing AtomsBase, an interface package for specifying atomic geometries. I think Fermi.jl would be a great candidate to support AtomsBase and aid interoperability with other atomistic tools in Julia!

I thought I'd put this on your radar, and would be happy to discuss the best way to go about implementing the interface, and/or contribute to a PR to do so!

@mfherbst
Copy link

mfherbst commented Mar 26, 2022

Quick comment: One thing I realised recently when thinking about the current state of AtomsBase is that for molecular systems a potential nuisance could be that we don't yet have custom system-level properties.
Point is that charge/multiplicity are usually stored on the level of a full system for molecular structures (in contrast to solid-state systems where this is more commonly dealt with by atom-specific magnetic moments / charges). Not sure this is a blocker, but something one should probably fix in AtomsBase first if there is interest on the maintainer side of Fermi.jl to integrate with AtomsBase.

@gustavojra
Copy link
Member

Hi there,

Sorry about the delay. Fermi currently uses Molecules.jl to handle description of atoms, more specifically handling XYZ type data. Do you guys have some minimal examples of AtomsBase usage? I could make it fit into Molecules.jl.

@rkurchin
Copy link
Author

rkurchin commented Apr 7, 2022

Ah, I didn't notice that, my apologies. Probably Molecules.jl would be the right place then; I should have filed this issue over there. For more details on AtomsBase you can see the docs. At a quick glance, I think the way to do it would be to make your Molecule type a subtype of AbstractSystem and dispatch the interface functions on that by pretty trivial analogy with our FlexibleSystem example.

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

3 participants