remove useless code

Author: haozhihan

source/module_lr/utils/lr_util.hpp

@@ -109,12 +109,6 @@ namespace LR_Util
         assert(psi_kfirst.get_nk() == 1);
         assert(nk_in * nbasis_in == psi_kfirst.get_nbasis());
 
-        // std::vector<int> ngk_vector(nk_in, 0);
-        // for (size_t i = 0; i < ngk_vector.size(); i++)
-        // {
-        //     ngk_vector[i] = psi_kfirst.get_ngk_pointer()[i];
-        // }
-
         int ib_now = psi_kfirst.get_current_b();
         psi_kfirst.fix_b(0);    // for get_pointer() to get the head pointer
         psi::Psi<T, Device> psi_bfirst(psi_kfirst.get_pointer(), nk_in, psi_kfirst.get_nbands(), nbasis_in, false);
@@ -129,12 +123,6 @@ namespace LR_Util
         int ib_now = psi_bfirst.get_current_b();
         int ik_now = psi_bfirst.get_current_k();
 
-        // std::vector<int> ngk_vector(psi_bfirst.get_nk(), 0);
-        // for (size_t i = 0; i < ngk_vector.size(); i++)
-        // {
-        //     ngk_vector[i] = psi_bfirst.get_ngk_pointer()[i];
-        // }
-
         psi_bfirst.fix_kb(0, 0);    // for get_pointer() to get the head pointer
         psi::Psi<T, Device> psi_kfirst(psi_bfirst.get_pointer(), 1, psi_bfirst.get_nbands(), psi_bfirst.get_nk() * psi_bfirst.get_nbasis(), true);
         psi_bfirst.fix_kb(ik_now, ib_now);

source/module_psi/psi.cpp

@@ -44,13 +44,6 @@ Psi<T, Device>::~Psi()
     }
 }
 
-// template <typename T, typename Device> Psi<T, Device>::Psi(const int* ngk_in)
-// {
-//     this->ngk = ngk_in;
-//     this->npol = PARAM.globalv.npol;
-//     this->device = base_device::get_device_type<Device>(this->ctx);
-// }
-
 template <typename T, typename Device>
 Psi<T, Device>::Psi(const int nk_in, const int nbd_in, const int nbs_in, const int* ngk_in, const bool k_first_in)
 {
@@ -70,40 +63,16 @@ Psi<T, Device>::Psi(const int nk_in, const int nbd_in, const int nbs_in, const i
 }
 
 // Constructor 8-1:
-// template <typename T, typename Device>
-// Psi<T, Device>::Psi(T* psi_pointer,
-//                     const int nk_in,
-//                     const int nbd_in,
-//                     const int nbs_in,
-//                     const int* ngk_in,
-//                     const bool k_first_in)
-// {
-//     this->k_first = k_first_in;
-//     this->ngk = ngk_in;
-//     this->current_b = 0;
-//     this->current_k = 0;
-//     this->npol = PARAM.globalv.npol;
-//     this->device = base_device::get_device_type<Device>(this->ctx);
-//     this->nk = nk_in;
-//     this->nbands = nbd_in;
-//     this->nbasis = nbs_in;
-//     this->current_nbasis = nbs_in;
-//     this->psi_current = this->psi = psi_pointer;
-//     this->allocate_inside = false;
-//     // Currently only GPU's implementation is supported for device recording!
-//     base_device::information::print_device_info<Device>(this->ctx, GlobalV::ofs_device);
-// }
-
-// Constructor 8-3:
 template <typename T, typename Device>
 Psi<T, Device>::Psi(T* psi_pointer,
                     const int nk_in,
                     const int nbd_in,
                     const int nbs_in,
+                    const std::vector<int>& ngk_vector_in,
                     const bool k_first_in)
 {
     this->k_first = k_first_in;
-    this->ngk = nullptr;
+    this->ngk = ngk_vector_in.data();
     this->current_b = 0;
     this->current_k = 0;
     this->npol = PARAM.globalv.npol;
@@ -120,15 +89,10 @@ Psi<T, Device>::Psi(T* psi_pointer,
 
 // Constructor 8-2:
 template <typename T, typename Device>
-Psi<T, Device>::Psi(T* psi_pointer,
-                    const int nk_in,
-                    const int nbd_in,
-                    const int nbs_in,
-                    const std::vector<int>& ngk_vector_in,
-                    const bool k_first_in)
+Psi<T, Device>::Psi(T* psi_pointer, const int nk_in, const int nbd_in, const int nbs_in, const bool k_first_in)
 {
     this->k_first = k_first_in;
-    this->ngk = ngk_vector_in.data();
+    this->ngk = nullptr;
     this->current_b = 0;
     this->current_k = 0;
     this->npol = PARAM.globalv.npol;

source/module_psi/psi.h

@@ -58,14 +58,6 @@ class Psi
     Psi(const Psi<T_in, Device_in>& psi_in);
 
     // Constructor 8-1: a pointer version of constructor 3
-    // Psi(T* psi_pointer,
-    //     const int nk_in,
-    //     const int nbd_in,
-    //     const int nbs_in,
-    //     const int* ngk_in = nullptr,
-    //     const bool k_first_in = true);
-
-    // Constructor 8-2: a pointer version of constructor 3
     // only used in hsolver-pw function pointer.
     Psi(T* psi_pointer,
         const int nk_in,
@@ -74,7 +66,7 @@ class Psi
         const std::vector<int>& ngk_vector_in,
         const bool k_first_in = true);
 
-    // Constructor 8-3: a pointer version of constructor 3
+    // Constructor 8-2: a pointer version of constructor 3
     // only used in operator.cpp call_act func
     Psi(T* psi_pointer,
         const int nk_in,