search

zeratax / yacx

Yet Another CudaExecutor - wrapper to easily compile and execute cuda kernels
https://zeratax.github.io/yacx
MIT License
8 stars 4 forks source link

Consistent naming scheme / Design #2

Closed zeratax closed 4 years ago

zeratax commented 5 years ago 
.instantiate(type_of(data), 5)
.instantiate<float, std::integral_constant<int, 5>>()
hadis1000 commented 4 years ago

Ich bin fuer mvar. So kann man in der ganzen datei nach m suchen um alle member variablen zu finden.

hadis1000 commented 4 years ago

Nach der umbennung von Kernel und Program sollten die args jetzt KernelArgs heissen.

hadis1000 commented 4 years ago

Meinungen?

zeratax commented 4 years ago

+1 m_var +1 KernelArg

zeratax commented 4 years ago

.instantiate(type_of(data), 5) +1 Ich glaube das kann man dann einfach für is_arithmetic und strings instantieren und dann gut linken.

Edit: Okay eig gehen variadic funktionen wohl nie als library nehme ich an. Trotzdem finde ich das Format am schönsten. Muss man halt dann für einen string initialisieren, e.g. .initialize("int, int 5"}, wenn man es dann im executor verwenden will