Skip to content
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

replace cal_stress in GPU relax calculation,use NUM_STREAM in INPUT file and fix memory leak in force calculation #4200

Merged
merged 42 commits into from
May 27, 2024

Conversation

A-006
Copy link
Collaborator

@A-006 A-006 commented May 22, 2024

When the relaxation is performed, it is mandotory to use the cal_stresss parameter in the current version of ABACUS. However, it may not be necessary to compute the stress. The calculation using the pipeline needs to be controlled via the NUM_STREAM parameter, so add this parameter.
When calculating force, a memory leak occurred. This issue was resolved, and an instance of previously unreleased memory was also identified and addressed.

Linked Issue

Fix #4199

@A-006 A-006 closed this May 22, 2024
@A-006 A-006 reopened this May 22, 2024
@A-006 A-006 closed this May 22, 2024
@A-006 A-006 reopened this May 22, 2024
@A-006 A-006 changed the title replace cal_stress in GPU relax calculation use NUM_STREAM in INPUT file replace cal_stress in GPU relax calculation and use NUM_STREAM in INPUT file May 23, 2024
docs/advanced/input_files/input-main.md Show resolved Hide resolved
source/module_hamilt_lcao/module_gint/gint.cpp Outdated Show resolved Hide resolved
source/module_hamilt_lcao/module_gint/gint_force.h Outdated Show resolved Hide resolved
source/module_hamilt_lcao/module_gint/gint_force.h Outdated Show resolved Hide resolved
source/module_hamilt_lcao/module_gint/gint_force_gpu.cu Outdated Show resolved Hide resolved
source/module_hamilt_lcao/module_gint/gint_force_gpu.cu Outdated Show resolved Hide resolved
@mohanchen mohanchen added the Bugs (Exclude input and output) Bugs that only solvable with sufficient knowledge of DFT label May 24, 2024
@A-006 A-006 changed the title replace cal_stress in GPU relax calculation and use NUM_STREAM in INPUT file replace cal_stress in GPU relax calculation,use NUM_STREAM in INPUT file and fix memory leak in force calculation May 24, 2024
@mohanchen mohanchen merged commit 992ad9b into deepmodeling:develop May 27, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugs (Exclude input and output) Bugs that only solvable with sufficient knowledge of DFT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use cal_stress in GPU Relax calculation
2 participants