Skip to content

Commit 597d012

Browse files
committed
modify the mpi_flag_ as flase
1 parent eb419aa commit 597d012

File tree

10 files changed

+9
-27
lines changed

10 files changed

+9
-27
lines changed

source/module_basis/module_pw/pw_basis.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ class PW_Basis
101101
//===============================================
102102
public:
103103
#ifdef __MPI
104-
MPI_Comm pool_world;
104+
MPI_Comm pool_world=MPI_COMM_NULL;
105105
#endif
106106

107107
int *ig2isz=nullptr; // map ig to (is, iz).
@@ -437,7 +437,7 @@ class PW_Basis
437437

438438
std::string device = "cpu"; ///< cpu or gpu
439439
std::string precision = "double"; ///< single, double, mixing
440-
bool mpi_flag_ = true; ///< ture,is use mpi or not
440+
bool mpi_flag_ = false; ///< ture,is use mpi or not
441441
bool double_data_ = true; ///< if has double data
442442
bool float_data_ = false; ///< if has float data
443443
};

source/module_basis/module_pw/pw_init.cpp

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,21 +10,12 @@ void PW_Basis:: initmpi(
1010
MPI_Comm pool_world_in
1111
)
1212
{
13-
if (mpi_flag_)
14-
{
1513
this->poolnproc = poolnproc_in;
1614
this->poolrank = poolrank_in;
1715
this->pool_world = pool_world_in;
18-
}else
19-
{
20-
ModuleBase::WARNING_QUIT("PW_Basis","to use MPI_ FFT, please set the mpi_flag as true");
21-
}
16+
mpi_flag_ = ((this->poolnproc>1) || !(this->pool_world == MPI_COMM_NULL));
2217
}
2318
#endif
24-
void PW_Basis::set_mpi(const bool mpi_flag_in)
25-
{
26-
this->mpi_flag_ = mpi_flag_in;
27-
}
2819
///
2920
/// Init the grids for FFT
3021
/// Input: lattice vectors of the cell, Energy cut off for G^2/2

source/module_basis/module_pw/test_gpu/pw_basis_C2C.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ TEST_F(PWTEST, recip_to_real_C2C_double)
2424
bool xprime = false;
2525

2626
// init
27-
pwtest.set_mpi(false);
27+
pwtest.initmpi(1,0,MPI_COMM_NULL);
2828
pwtest.initgrids(lat0, latvec, wfcecut);
2929
pwtest.initparameters(gamma_only, wfcecut, distribution_type, xprime);
3030
pwtest.setuptransform();
@@ -161,7 +161,7 @@ TEST_F(PWTEST, recip_to_real_C2C_float)
161161
int distribution_type = 1;
162162
bool xprime = false;
163163

164-
pwtest.set_mpi(false);
164+
pwtest.initmpi(1,0,MPI_COMM_NULL);
165165
pwtest.initgrids(lat0, latvec, wfcecut);
166166
pwtest.initparameters(gamma_only, wfcecut, distribution_type, xprime);
167167
pwtest.setuptransform();

source/module_basis/module_pw/test_gpu/pw_basis_C2R.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ TEST_F(PWTEST, recip_to_real_double)
2727
bool xprime = false;
2828

2929
// init
30-
pwtest.set_mpi(false);
30+
pwtest.initmpi(1,0,MPI_COMM_NULL);
3131
pwtest.initgrids(lat0, latvec, wfcecut);
3232
pwtest.initparameters(gamma_only, wfcecut, distribution_type, xprime);
3333
pwtest.setuptransform();
@@ -164,7 +164,7 @@ TEST_F(PWTEST, recip_to_real_float)
164164
int distribution_type = 1;
165165
bool xprime = false;
166166

167-
pwtest.set_mpi(false);
167+
pwtest.initmpi(1,0,MPI_COMM_NULL);
168168
pwtest.initgrids(lat0, latvec, wfcecut);
169169
pwtest.initparameters(gamma_only, wfcecut, distribution_type, xprime);
170170
pwtest.setuptransform();

source/module_basis/module_pw/test_gpu/pw_basis_k_C2C.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ TEST_F(PWTEST, pw_basis_k_C2C_double)
2424
int distribution_type = 1;
2525
bool xprime = false;
2626
//--------------------------------------------------
27-
pwtest.set_mpi(false);
2827
// init //real parameter
28+
pwtest.initmpi(1,0,MPI_COMM_NULL);
2929
pwtest.initgrids(lat0, latvec, 4 * wfcecut);
3030
pwtest.initparameters(gamma_only, wfcecut, nks, kvec_d, distribution_type, xprime);
3131
pwtest.setuptransform();
@@ -162,8 +162,8 @@ TEST_F(PWTEST, pw_basis_k_C2C_float)
162162
int distribution_type = 1;
163163
bool xprime = false;
164164
//--------------------------------------------------
165-
pwtest.set_mpi(false);
166165
// init //real parameter
166+
pwtest.initmpi(1,0,MPI_COMM_NULL);
167167
pwtest.initgrids(lat0, latvec, 4 * wfcecut);
168168
pwtest.initparameters(gamma_only, wfcecut, nks, kvec_d, distribution_type, xprime);
169169
pwtest.setuptransform();

source/module_basis/module_pw/test_serial/pw_basis_k_test.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,6 @@ class PWBasisKTEST: public testing::Test
3939
ModulePW::PW_Basis_K basis_k;
4040
void SetUp()
4141
{
42-
// basis_k= ModuleBase::PW_Basis_k(device_flag,preci)
43-
basis_k.set_mpi(false);
4442
}
4543
};
4644

source/module_basis/module_pw/test_serial/pw_basis_test.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,6 @@ class PWBasisTEST: public testing::Test
5050
ModulePW::PW_Basis pwb1;
5151
void SetUp()
5252
{
53-
pwb.set_mpi(false);
5453
}
5554
};
5655

source/module_elecstate/test/charge_mixing_test.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,12 +82,10 @@ class ChargeMixingTest : public ::testing::Test
8282
ChargeMixingTest()
8383
{
8484
// Init pw_basis
85-
pw_basis.set_mpi(false);
8685
pw_basis.initgrids(4, ModuleBase::Matrix3(1, 0, 0, 0, 1, 0, 0, 0, 1), 20);
8786
pw_basis.initparameters(false, 20);
8887
pw_basis.setuptransform();
8988
pw_basis.collect_local_pw();
90-
pw_dbasis.set_mpi(false);
9189
pw_dbasis.initgrids(4, ModuleBase::Matrix3(1, 0, 0, 0, 1, 0, 0, 0, 1), 40);
9290
pw_dbasis.initparameters(false, 40);
9391
pw_dbasis.setuptransform(&pw_basis);

source/module_elecstate/test/charge_test.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,6 @@ class ChargeTest : public ::testing::Test
8080
ucell = utp.SetUcellInfo();
8181
charge = new Charge;
8282
rhopw = new ModulePW::PW_Basis;
83-
rhopw->set_mpi(false);
8483
rhopw->initgrids(ucell->lat0, ucell->latvec, elecstate::tmp_gridecut);
8584
rhopw->distribute_r();
8685
rhopw->initparameters(false, elecstate::tmp_gridecut);

source/module_elecstate/test/potential_new_test.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ class PotentialNewTest : public ::testing::Test
117117
solvent = new surchem();
118118
etxc = new double;
119119
vtxc = new double;
120-
rhopw->set_mpi(false);
121120
elecstate::Set_GlobalV_Default();
122121
}
123122
virtual void TearDown()
@@ -579,7 +578,6 @@ TEST_F(PotentialNewTest, InterpolateVrsDoubleGrids)
579578
XC_Functional::func_type = 3;
580579
XC_Functional::ked_flag = true;
581580
// Init pw_basis
582-
rhodpw->set_mpi(false);
583581
rhopw->initgrids(4, ModuleBase::Matrix3(1, 0, 0, 0, 1, 0, 0, 0, 1), 4);
584582
rhopw->initparameters(false, 4);
585583
rhopw->setuptransform();
@@ -629,7 +627,6 @@ TEST_F(PotentialNewTest, InterpolateVrsWarningQuit)
629627
rhopw->collect_local_pw();
630628
rhodpw->gamma_only = false;
631629

632-
rhodpw->set_mpi(false);
633630
rhodpw->initgrids(4, ModuleBase::Matrix3(1, 0, 0, 0, 1, 0, 0, 0, 1), 6);
634631
rhodpw->initparameters(false, 6);
635632
static_cast<ModulePW::PW_Basis_Sup*>(rhodpw)->setuptransform(rhopw);

0 commit comments

Comments
 (0)