[DEBT] rename .h|.hpp|.cuh|.cu|.cpp files properly in our cuml C++ source base

(44 commenti) (0 reazioni) (1 assegnatario)Python (644 fork)auto 404

Build or DepCUDA / C++Tech Debtgood first issueproposal

Metriche repository

Currently, they are just all over the place! I propose the following rule-of-thumb:

If a header file contains cuda kernels and/or device methods, then it should be a .cuh.
Else, it should be a .hpp (if the current header file is already named as a .h, feel free to leave it as it is, in order to minimize the amount of ripple changes this will cause).
If a header file is used to declare our C interface, it should be a .h
If the header file includes a .cuh, then it should be a .cuh as well
If a source includes a .cuh, then it should be a .cu, else it should be a .cpp
All source files defining our C interface should be .c

This will certainly help us in reducing compilation time as well (if we end up cleaning some of the .cu files and renaming them to .cpp).

Direzione di ricerca: Identifica tutti i file con estensione .h, .hpp, .cuh, .cu, .cpp nel codice sorgente C++ di cuml. Ridenominali secondo le regole: (1) file con kernel CUDA diventano .cuh; (2) altrimenti .hpp (se già .h, lascia); (3) header dell'interfaccia C rimangono .h; (4) header che includono un .cuh diventano .cuh; (5) sorgenti che includono .cuh diventano .cu, altrimenti .cpp; (6) sorgenti dell'interfaccia C diventano .c. Dopo la ridenominazione, aggiorna tutte le direttive #include nel codice per corrispondere ai nuovi nomi. Usa Git per tracciare le modifiche e verifica la compilazione con il sistema di build.
Tech stack: ccpp
Dominio: backend
Tipo issue: Refactoring
Difficoltà: 3
Tempo stimato: 3-5 giorni
Stato attività: Attiva
Chiarezza: Chiara
Prerequisiti: C++Git
Adatta ai principianti: 60