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

No documentation for xml parameter file format(s) #50

Closed
max-veit opened this issue Feb 14, 2017 · 4 comments
Closed

No documentation for xml parameter file format(s) #50

max-veit opened this issue Feb 14, 2017 · 4 comments

Comments

@max-veit
Copy link
Contributor

It's incredibly frustrating as a new user of a potential to try to figure out what XML format will initialize it properly. For instance, with a FilePot, it currently seems impossible to specify a set of calc_args that will be used on every call, even though it seems there should be a straightforward way to do this in XML.

We need better documentation for how to go about setting up an XML parameter file for a potential in general, and some of the more complicated potentials specifically. The examples in share/Parameters are quite minimal and don't provide much useful info on how these things are actually parsed.

Normally I'd volunteer to write some of this documentation, but I still have no idea how the parsing for a general Potential xml stanza works.

@gabor1
Copy link
Contributor

gabor1 commented Feb 14, 2017 via email

@max-veit
Copy link
Contributor Author

isn't there some confusion of terminology here? the contents of the xml is used to initialise the potential. calc_args is something that the quip command line uses and appends it to the argument string on every call. if you find that a potential parses options in its calc call that you would like to set at initialisations, you need to edit the source, create new variables in the potential type, and add the parsing and setting of them in the init routine.

so i'm claiming that what you want is not actually calc_args.

Currently, there seems to be no other way to pass important command line arguments to a command called by FilePot. I suppose they could be included with the command itself, but drivers like castep_driver.py expect the arguments in a different order -- first xyzfile and outfile, then additional args -- which doesn't work with that approach.

So it looks like FilePot should be changed to accept a list of command line arguments that will always be added after xyzfile and outfile, since these (in cases like the template file) essentially define the potential. The calc_args can still be appended to the end as optional extras.

the reason there is no generic doc for xml is that every potential does its own parsing, and accepts different options.

Then maybe a good place to start would be to document how Potential (the base class) parses its XML stanza, what arguments it takes, and how this relates to the quippy or the command-line arguments for initializing the potential. This would then serve as a good introduction for figuring out how other potentials are initialized.

@gabor1
Copy link
Contributor

gabor1 commented Feb 16, 2017 via email

@max-veit
Copy link
Contributor Author

max-veit commented Mar 23, 2017

The documentation for xml initialisation of the most common potentials should probably be merged with the introductory IPython tutorial or other introductory quip/quippy tutorials -- see #59.

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