diff --git a/src/Sparse.cpp b/src/Sparse.cpp
index 4b6d8fe..11874ec 100644
--- a/src/Sparse.cpp
+++ b/src/Sparse.cpp
@@ -10,9 +10,9 @@ namespace Aurora
     {
     }
 
-    Sparse::Sparse(Matrix Cols, Matrix Rows, Matrix Values,size_t M, size_t N)
-    : mColIdxVector(Cols),
-    mRowIdxVector(Rows),
+    Sparse::Sparse(Matrix RowIdxs, Matrix ColIdxs , Matrix Values,size_t M, size_t N)
+    : mColIdxVector(ColIdxs),
+    mRowIdxVector(RowIdxs),
     mValueVector(Values),
     mM(M),
     mN(N)
diff --git a/src/Sparse.h b/src/Sparse.h
index 3c0f27a..e80b83e 100644
--- a/src/Sparse.h
+++ b/src/Sparse.h
@@ -7,13 +7,23 @@ namespace Aurora {
     {
     public:
         Sparse();
-        Sparse(Matrix Cols, Matrix Rows, Matrix Values,size_t M, size_t N);
+        Sparse(Matrix RowIdxs, Matrix ColIdxs, Matrix Values,size_t M, size_t N);
         ~Sparse();
         bool isValid() const;
         Matrix& getColVector() ;
         Matrix& getRowVector() ;
         Matrix& getValVector() ;
-        size_t getM() const ;
+        /**
+         * @brief Rows
+         * 
+         * @return size_t 
+         */
+        size_t getM() const;
+        /**
+         * @brief Cols
+         * 
+         * @return size_t 
+         */
         size_t getN() const;
         // TODO:add operators
     private: