-
Notifications
You must be signed in to change notification settings - Fork 142
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 ppconvert #874
Update ppconvert #874
Conversation
ye-luo
commented
May 25, 2018
- allows changing density mixing parameter via command line.
- update ppconvert manual section
@ye-luo thanks for adding more details on Could you add a few sentences on how a "reference state" should be chosen? |
@Paul-St-Young I added my limited knowledge of how to set the reference states. |
@ye-luo thanks a lot! |
The following script seems to produce a reasonable upf for silicon BIN=ppconvert pot=Si.TN cmd="$BIN --local_channel $loc --casino_pot pp.data --s_ref $ref --p_ref $ref --d_ref $ref" $cmd --upf $pot.upf >& convert_upf.out |
I should mention that my first instinct was to use |
The difference in orbitals may be caused by using a different XC functional or HF or DF. |
The converted pseudopotential can reproduce the Si band structure compared to the standard quantum espresso example (PP/examples/example01). I also tried manually replacing the <PP_PSWFC> and <PP_RHOATOM> entries in the UPF with data from awfn.data (<PP_R> and <PP_LOCAL> already exactly matches pp.data). I saw no change to the band structure. I think the small difference we saw is insignificant. |
@Paul-St-Young The incorrect number of nodes clearly indicates the wrong states are being solved for by ppconvert. You need to change the specification of the reference state. From an archived page (google search):
|
@Paul-St-Young Do you have enough information to convert the potentials that you need? Clearly (i) a set of converted potentials would avoid this problem and (ii) a better write up is needed for ppconvert to be useful or an alternative found. |
@prckent I looked at the archived page when I wrote my guidance for reference states. The following requirement is no more needed.
|
@prckent yes, I was able to obtain the pseudopotential I needed using the bash script I posted above. i.e. |
I also noticed that the starting charge in QE is off by 4pi If I scale the <PP_RHOATOM> entry in the generated UPF by 4pi, then the starting charge is almost correct The QE description of the UPF file says <PP_RHOATOM> should hold |
If all the pseudopotential files are created by ppconvert, this is not a big issue since QE will rescale them all. If mixed pseudopotential files (from OPIUM and BFD) are used, the starting density may be pretty bad. So fixing the density normalization is needed. |
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.
I updated the ghost instructions.