NCAR / micm

A model-independent chemistry module for atmosphere models
https://ncar.github.io/micm/
Apache License 2.0
5 stars 5 forks source link

The name of CUDA matrix class may be confusing #442

Closed sjsprecious closed 5 months ago

sjsprecious commented 5 months ago

Currently we have CudaVectorMatrix and CudaSparseMatrix class defined for the GPU code. However, an object of type CudaSparseMatrix is also a vector matrix. Thus what we really mean for the GPU code is actually CudaDenseVectorMatrix and CudaSparseVectorMatrix.

After discussion with @mattldawson and @mwaxmonsky , we decide to rename all the CudaVectorMatrix to CudaDenseMatrix for clarity. We will also drop the term Vector for all the GPU code, assuming that we will always use a vectorized matrix for GPU computing.