Skip to content

Test required for new labels in DeePKS. #6107

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.

Sign up for GitHub

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

Jump to bottom
Closed
ErjieWu opened this issue Apr 3, 2025 · 1 comment · Fixed by #6140
Closed

Test required for new labels in DeePKS. #6107

ErjieWu opened this issue Apr 3, 2025 · 1 comment · Fixed by #6140
Assignees
@ErjieWu
Labels
DeePKS Issues related to the DeePKS Unit Tests/Integreate Tests Issues/PR related to unit tests and integrate tests

Comments

@ErjieWu
Copy link
Collaborator

ErjieWu commented Apr 3, 2025 •
edited by mohanchen
Loading

Describe the Testing Issue

Currently, the unit test in DeePKS contains only energy&force&stress labels related checks. Unit test for functions related to orbital&Hamiltonian labels lacks.

Also, integrate tests for multi-k Hamiltonian label has not been added.
@ErjieWu ErjieWu self-assigned this Apr 3, 2025
@ErjieWu ErjieWu added the DeePKS Issues related to the DeePKS label Apr 3, 2025
@mohanchen
Copy link
Collaborator

That's an excellent idea!

@mohanchen mohanchen added the Unit Tests/Integreate Tests Issues/PR related to unit tests and integrate tests label Apr 8, 2025
@ErjieWu ErjieWu mentioned this issue Apr 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ErjieWu ErjieWu

Labels
DeePKS Issues related to the DeePKS Unit Tests/Integreate Tests Issues/PR related to unit tests and integrate tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Test: Add unit tests and integrate tests for new features of DeePKS. ErjieWu/abacus-develop
2 participants
@mohanchen @ErjieWu