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

Suggestion: Header for thermo.out file #473

Open
erhart1 opened this issue Aug 15, 2023 · 2 comments
Open

Suggestion: Header for thermo.out file #473

erhart1 opened this issue Aug 15, 2023 · 2 comments

Comments

@erhart1
Copy link
Collaborator

erhart1 commented Aug 15, 2023

It would be useful to include a header in the thermo.out file that specifies the meaning of the columns.
If these lines are prepended by a widely used comment symbols (e.g., #), the user experience should be (hopefully) unaffected.
At the same time it would help users (and parsers) to read the content of the file.
This would in particular improve readability/interpretability if the number of columns/data changes, e.g., when switching from an orthorhombic to a triclinic cell.
Finally, this would also be useful if one were to decides to add more information to this file in the future.

@erhart1 erhart1 mentioned this issue Aug 15, 2023
@hityingph
Copy link
Collaborator

Yes, I agree with. Acutally, I also suggested this before, however, Zheyong @brucefan1983 prefered the pure arrays output without strings (header).

@brucefan1983
Copy link
Owner

Perhaps we can do this for all the output files in GPUMD-v4.0, for which we will introduce breaking changes?

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

3 participants