rapidsai / raft

RAFT contains fundamental widely-used algorithms and primitives for machine learning and information retrieval. The algorithms are CUDA-accelerated and form building blocks for more easily writing high performance applications.
https://docs.rapids.ai/api/raft/stable/
Apache License 2.0
683 stars 180 forks source link

Correct initializer list order found by cuvs #2317

Closed robertmaynard closed 1 month ago

robertmaynard commented 1 month ago

Align our constuctor order to match what C++ spec says will occur:

Then, non-static data members are initialized in the order they were declared in the class definition (again regardless of the order of the mem-initializers).

Otherwise consumer of raft will get warnings when building with -Wall -Wextra and using the index type.

cjnolet commented 1 month ago

/merge