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

PDOS xml file compatibility issues #99

Open
albgar opened this issue Feb 26, 2021 · 0 comments
Open

PDOS xml file compatibility issues #99

albgar opened this issue Feb 26, 2021 · 0 comments

Comments

@albgar
Copy link
Member

albgar commented Feb 26, 2021

Currently Siesta produces two PDOS files in XML:

  • SystemLabel.PDOS is created by simple fortran write statements, and the relevant data is mostly in separate lines. This has traditionally made it useful for some third-party utilities such as Andrei Postnikov's post-processor.
  • SystemLabel.PDOS.xml is created by calls to the wxml module of xmlf90. It is more compact, and the code to generate it is simpler. However, due to an oversight, it is missing the 'norbitals' element. Otherwise its data content is the same as that of the PDOS file.

We do not like to have to maintain two different files, so we are planning to deprecate the PDOS file. This has consequences for the PDOS support in the plugin. To maintain compatibility with all versions of Siesta:

  • One should maybe check for the existence of both PDOS and PDOS.xml.
  • If the 'norbitals' element is not found, the value of n_orbitals can be simply inferred by counting the number of 'orbital' elements in the file.
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

1 participant