Skip to content

Update README to reflect template-based multi-scalar type support - #5

Merged
deltaeecs merged 2 commits into
masterfrom
copilot/update-readme-details
Dec 4, 2025
Merged

Update README to reflect template-based multi-scalar type support#5
deltaeecs merged 2 commits into
masterfrom
copilot/update-readme-details

Conversation

Copilot AI commented Dec 4, 2025

Copy link
Copy Markdown
Contributor

PR #4 converted the library to C++ templates, enabling all four scalar types (float, double, complex<float>, complex<double>) simultaneously without recompilation. The README still documented the old CSR4MPI_VALUE_TYPE compile-time switch approach.

Changes

  • Project description: Emphasize template-based design; remove "冷切换" (cold-switch) terminology
  • Features section: Add template classes, type aliases (cCSRMatrixF/D/CF/CD), and type traits (is_complex_v, is_supported_scalar_v, real_type_t)
  • Build section: Remove CSR4MPI_VALUE_TYPE parameter; note that all types are available in single build
  • New "标量类型使用" section: Type alias table, usage examples, type traits documentation
  • Directory overview: Update to reflect templated headers and new test_all_scalar_types.cpp

Usage Example

using namespace csr4mpi;

// All types available simultaneously
cCSRMatrixD matDouble(0, n, n, rowPtr, colInd, valuesDouble);
cCSRMatrixCF matComplexFloat(0, n, n, rowPtr, colInd, valuesComplexFloat);

std::vector<double> xD(n, 1.0), yD;
SpMV(matDouble, xD, yD);
Original prompt

根据上一次commit更新的内容,更新 readme,稍微详细一点


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: deltaeecs <48714525+deltaeecs@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README with detailed information Update README to reflect template-based multi-scalar type support Dec 4, 2025
Copilot AI requested a review from deltaeecs December 4, 2025 07:16
@deltaeecs
deltaeecs marked this pull request as ready for review December 4, 2025 07:33
@deltaeecs
deltaeecs merged commit 3823faf into master Dec 4, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants