[Quavitor]

#codereview #programowanie #cuda

Może ktoś rzucić okiem czy mam poprawnie zaimplementowany dithering?

http://pastebin.com/XD1qRp7q

[p.....y]

@Quavitor: nie znam algorytmu, ale jeśli chodzi o sam kod - szybkie google mówi, że

__device__

to CUDA, czyli zakładam, że ma być szybko?

[badasq]

@Quavitor: @paffnucy:

ad tip #1

1.

[p.....y]

@Quavitor: Nie słuchaj mnie, słuchaj @badasq, nie wiem co we łbie miałem pisząc komentarz wyżej. Oczywiście, że przesunięciem bitowym można zastąpić dzielenie, ale nie modulo (btw. dziwne, że z trzech osób plusujących mój komentarz, nikt nie zwrócił uwagi na tak podstawowy błąd :))

[Quavitor]

@paffnucy: @badasq:

Tak, ma być szybko ale najpierw ma być poprawnie :)

Dzielenie przez 3 jest po to aby obliczyć średnią (w kodzie nie ma x, @badasq skrócił zapis), dzielenie przez 15 bo tam wymaga algo (o ile dobrze go rozumiem).

Dzięki za odpowiedzi, jeśli są jeszcze jakieś protipy to zachęcam ;)

[badasq]

@Quavitor: jako, że mam więcej czasu to otwarłem kod i

linia 1: FF(a) ((int) a ) kuuuuurwa, co to? Dlaczego akurat FF, a może lepiej JJ albo IC? Sorry, ale nie cierpię makr i języka C ;-) w każdym razie nazwy powinny w kodzie mieć jakieś znaczenie, najlepiej żeby nazwy metod mówiły, co dana metoda robi w sposób wyczerpujący, FF nic mi nie mówi,

linia 13&14: pomijając fakt, że są one zbędne,