Skip to content

Test: Add RT-TDDFT GPU integrate tests and fix some related bugs/typos #6201

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

Merged
merged 6 commits into from
May 10, 2025

Conversation

AsTonyshment
Copy link
Collaborator

@AsTonyshment AsTonyshment commented May 8, 2025

Linked Issue

Fix #6200

What's changed?

  • Added RT-TDDFT GPU integrate tests.
  • Refactored RT-TDDFT CPU test files.
  • Fixed some typos, including: 002_NO_H20_GPU -> 002_NO_H2O_GPU, propogator -> propagator, gague -> gauge, etc.

Notice

Test case 07_NO_EDM_TDDFT has been temporarily commented out because the GPU results are inconsistent with the CPU results. See issue #6199 for details.

@AsTonyshment AsTonyshment requested a review from mohanchen May 9, 2025 02:24
@mohanchen mohanchen added the Unit Tests/Integreate Tests Issues/PR related to unit tests and integrate tests label May 10, 2025
@mohanchen mohanchen merged commit eb20d48 into deepmodeling:develop May 10, 2025
14 checks passed
@AsTonyshment AsTonyshment deleted the add_rttd_gpu_testcase branch May 10, 2025 16:39
kluophysics pushed a commit to kluophysics/abacus-develop that referenced this pull request May 11, 2025
deepmodeling#6201)

* Add RT-TDDFT GPU test cases and fix some bugs

* Modify GitHub CI/CD config file

* Trigger ReadTheDocs rebuild

* Trigger ReadTheDocs rebuild

* Delete useless info in result.ref
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Unit Tests/Integreate Tests Issues/PR related to unit tests and integrate tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need to add GPU CI/CD integration tests for RT-TDDFT
2 participants