Commit 48639db
abacus_fixer
refactor(dftu): move DFT+U operator from module_operator_lcao to module_dftu
Move DFTU/OperatorDFTU<OperatorLCAO> operator implementation files
(dftu.hpp, dftu_lcao.h/cpp, dftu_fs.cpp, op_dftu_lcao.h/cpp) and the
related unit test (test_dftu.cpp -> dftu_lcao_test.cpp) from
module_operator_lcao to module_dftu. This is a step toward removing
module_operator_lcao and grouping DFT+U code by physical domain.
CMake changes:
- module_dftu CMakeLists adds the 3 cpp files guarded by ENABLE_LCAO,
since module_dftu is unconditionally added by source_lcao while
these files depend on operator_lcao.h
- module_operator_lcao CMakeLists drops the 3 cpp files
- source_lcao CMakeLists drops the 3 cpp entries from the hamilt_lcao
OBJECT library (now provided by the dftu OBJECT library)
- New dftu_lcao_test target guarded by ENABLE_LCAO AND ENABLE_MPI
matches the original MODULE_LCAO_operator_dftu_test build condition
Include path updates in 5 external call sites plus op_dftu_lcao.h
internal operator_lcao.h path (changed from same-directory relative
to full path since the file is no longer co-located).
Verified: cmake configure, make abacus_max_para + dftu_lcao_test,
and ctest dftu_pw_test/dftu_core_test/dftu_operator_test/dftu_lcao_test
all pass; agent_governance_check reports only include-path change
warnings with no errors.1 parent 22659fc commit 48639db
16 files changed
Lines changed: 31 additions & 22 deletions
File tree
- source
- source_io/module_hs
- source_lcao
- module_dftu
- test
- module_operator_lcao
- test
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
19 | 18 | | |
20 | 19 | | |
21 | 20 | | |
| |||
32 | 31 | | |
33 | 32 | | |
34 | 33 | | |
35 | | - | |
36 | | - | |
37 | 34 | | |
38 | 35 | | |
39 | 36 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | | - | |
| 34 | + | |
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | | - | |
| 39 | + | |
40 | 40 | | |
41 | 41 | | |
42 | 42 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
11 | 19 | | |
12 | 20 | | |
13 | 21 | | |
| |||
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
0 commit comments