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

Provide pre-packaged .ion files as input to a calculation #26

Open
albgar opened this issue May 29, 2019 · 3 comments
Open

Provide pre-packaged .ion files as input to a calculation #26

albgar opened this issue May 29, 2019 · 3 comments

Comments

@albgar
Copy link
Member

albgar commented May 29, 2019

As an alternative to using a pseudopotential+basis-set-spec pair, we could use pre-packaged .ion files as input to calculations.

This has some benefits for "production calculations" and also adds some more flexibility to the users. For example, a user of an old version of Siesta could still benefit from PSML pseudopotentials by using an .ion file produced by a modern version.

Production of "ion files" might be optional in a calculation, be it a full Siesta calculation or a simpler one with a "gen-basis" program, for example.

Sets of .ion files could be grouped into 'families'.

More metadata than is currently available in an .ion file could be added to the node.

@bosonie
Copy link
Member

bosonie commented Jun 4, 2019

Oh wow! I didn't even know there was this possibility of passing .ion files to Siesta instead of Pseudopotential and basis. Good to know!
To create .ion families is a good idea, although we have to remind our-self and to the users that the definition of "good basis set" is system dependent. I mean that if a basis choice has been tested on the Mg crystal element, it is not guaranteed to work well for calculations on MgO.
To facilitate basis set testing with aiida is something I have in the back of my mind since long time and I would see this as a priority compered to the issue described here, but that's just my opinion.

@bosonie
Copy link
Member

bosonie commented Dec 13, 2020

This issue is partially addressed by #96, that created the IonData class. Still missing:

  • Accepting the IonData as inputs of a calculation
  • Creation of IonData families (this part needs to be discussed in depth)

@bosonie
Copy link
Member

bosonie commented Mar 19, 2021

Another point addressed, in #100:

  • Accepting the IonData as inputs of a calculation
  • Creation of IonData families (this part needs to be discussed in depth)

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

No branches or pull requests

2 participants