Skip to content

Commit 8a3154e

Browse files
committed
make nk non-optional
1 parent a8c1164 commit 8a3154e

File tree

5 files changed

+7
-7
lines changed

5 files changed

+7
-7
lines changed

source/module_elecstate/module_dm/density_matrix.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ DensityMatrix<TK, TR>::~DensityMatrix()
2626

2727
template <typename TK, typename TR>
2828
DensityMatrix<TK, TR>::DensityMatrix(const Parallel_Orbitals* paraV_in, const int nspin, const std::vector<ModuleBase::Vector3<double>>& kvec_d, const int nk)
29-
: _paraV(paraV_in), _nspin(nspin), _kvec_d(kvec_d), _nk((nk > 0 && nk <= _kvec_d.size()) ? nk : _kvec_d.size())
29+
: _paraV(paraV_in), _nspin(nspin), _kvec_d(kvec_d), _nk(nk)
3030
{
3131
ModuleBase::TITLE("DensityMatrix", "DensityMatrix-MK");
3232
const int nks = _nk * _nspin;

source/module_elecstate/module_dm/density_matrix.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ class DensityMatrix
5151
* @param nk number of k-points, not always equal to K_Vectors::get_nks()/nspin_dm.
5252
* if remains default or large than kvec_d.size(), it will be set to kvec_d.size()
5353
*/
54-
DensityMatrix(const Parallel_Orbitals* _paraV, const int nspin, const std::vector<ModuleBase::Vector3<double>>& kvec_d, const int nk = -1);
54+
DensityMatrix(const Parallel_Orbitals* _paraV, const int nspin, const std::vector<ModuleBase::Vector3<double>>& kvec_d, const int nk);
5555

5656
/**
5757
* @brief Constructor of class DensityMatrix for gamma-only calculation, where kvector is not required

source/module_elecstate/module_dm/test/test_dm_R_init.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ TEST_F(DMTest, DMInit1)
118118
// construct DM
119119
std::cout << "dim0: " << paraV->dim0 << " dim1:" << paraV->dim1 << std::endl;
120120
std::cout << "nrow: " << paraV->nrow << " ncol:" << paraV->ncol << std::endl;
121-
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d);
121+
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d, nks);
122122
// initialize this->_DMR
123123
Grid_Driver gd(0,0);
124124
DM.init_DMR(&gd, &ucell);
@@ -145,7 +145,7 @@ TEST_F(DMTest, DMInit2)
145145
// construct DM
146146
std::cout << "dim0: " << paraV->dim0 << " dim1:" << paraV->dim1 << std::endl;
147147
std::cout << "nrow: " << paraV->nrow << " ncol:" << paraV->ncol << std::endl;
148-
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d);
148+
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d, nks);
149149
// initialize Record_adj using Grid_Driver
150150
Grid_Driver gd(0,0);
151151
Record_adj ra;

source/module_elecstate/module_dm/test/test_dm_constructor.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ TEST_F(DMTest, DMConstructor_nspin1)
127127
std::cout << "dim0: " << paraV->dim0 << " dim1:" << paraV->dim1 << std::endl;
128128
std::cout << "nrow: " << paraV->nrow << " ncol:" << paraV->ncol << std::endl;
129129
int nspin = 1;
130-
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d);
130+
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d, nks);
131131
// compare
132132
EXPECT_EQ(DM.get_DMK_nks(), kv->get_nks());
133133
EXPECT_EQ(DM.get_DMK_nrow(), paraV->nrow);

source/module_elecstate/module_dm/test/test_dm_io.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ TEST_F(DMTest, DMConstructor1)
142142
int nspin = 1;
143143
// construct DM
144144
std::cout << paraV->nrow << paraV->ncol << std::endl;
145-
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d);
145+
elecstate::DensityMatrix<double, double> DM(paraV, nspin, kv->kvec_d, kv->get_nks());
146146
// read DMK
147147
std::string directory = "./support/";
148148
for (int is = 1; is <= nspin; ++is)
@@ -162,7 +162,7 @@ TEST_F(DMTest, DMConstructor1)
162162
}
163163
}
164164
// construct a new DM
165-
elecstate::DensityMatrix<double, double> DM1(paraV, nspin, kv->kvec_d);
165+
elecstate::DensityMatrix<double, double> DM1(paraV, nspin, kv->kvec_d, kv->get_nks());
166166
directory = "./support/output";
167167
for (int is = 1; is <= nspin; ++is)
168168
{

0 commit comments

Comments
 (0)