-
Notifications
You must be signed in to change notification settings - Fork 118
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
Feature : Add deepks_v_delta, which can help to train DeepKS model with loss term about Hamiltonian, psi and band. #2806
Conversation
…rm about Hamiltonian, psi and band. When deepks_out_labels equals 1 , it can output labels about Hamiltonian. Meanwhile, it will output v_delta_precalc when deepks_v_delta equals 1, and output psialpha and grad_evdm to save disk memory when deepks_v_delta equal 2. Add tests for deepks_v_delta equals 1 and 2.
…rm about Hamiltonian, psi and band. When deepks_out_labels equals 1 , it can output labels about Hamiltonian. Meanwhile, it will output v_delta_precalc when deepks_v_delta equals 1, and output psialpha and grad_evdm to save disk memory when deepks_v_delta equal 2. Add tests for deepks_v_delta equals 1 and 2.
Looks good to me. |
@@ -362,4 +382,56 @@ void LCAO_Deepks::del_orbital_pdm_shell(const int nks) | |||
return; | |||
} | |||
|
|||
void LCAO_Deepks::init_v_delta_pdm_shell(const int nks,const int nlocal) |
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.
Please add unit test for newly added functions.
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.
New integrate tests has covered the function.
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.
There are many newly added functions, they will reduce the coverage rate of unit tests.
@@ -362,4 +382,56 @@ void LCAO_Deepks::del_orbital_pdm_shell(const int nks) | |||
return; | |||
} | |||
|
|||
void LCAO_Deepks::init_v_delta_pdm_shell(const int nks,const int nlocal) |
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.
There are many newly added functions, they will reduce the coverage rate of unit tests.
Will PR again in the near future |
#2805