-
Notifications
You must be signed in to change notification settings - Fork 177
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
update abacus-scf generator #575
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your improvement! I only have one question left by @amcadmus in previous PR's conservation. "What is the reason of removing "pw"? does the current implementation support both pw and atomic basis?"
Yes. I examined the output of SCF and MD output format of both pw and lcao, and both of them are in the same format. Therefore I removed the 'pw' tag. |
@@ -131,17 +131,29 @@ | |||
"fp_task_max": 30, | |||
"fp_task_min": 8, | |||
"fp_pp_path": ".", | |||
"fp_pp_files": [ "H_HSCV_PBE-1.0.UPF","C_HSCV_PBE-1.0.UPF"], | |||
"fp_pp_files": [ "./H_HSCV_PBE-1.0.UPF","./C_HSCV_PBE-1.0.UPF"], | |||
"user_fp_params":{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may consider generating abacus input from a template file in a future PR.
* update abacus_scf * add new parameters in test cases * add assertion info
Add some parameters in generating INPUT and STRU file for ABACUS.