Grafika - Észrevételek, javaslatok

A7IO6D

• Az órai munka rendben van.
• A repository-ba egy README.md-be írni kellene a féléves feladattal kapcsolatban.

A86ANU

• A repository-ba a README.md-be írni kellene a féléves feladattal kapcsolatban.
• A féléves feladatot külön jegyzékbe kellene szervezni.
• A main.c lényegesen rövidebb kellene, hogy legyen.
• A bufferek méretét nevesített konstansokba kellene kiemelni.
• A dokumentációs kommentek elegendő, hogy ha a header-ökben vannak. (Ott viszont lehetnek részletesebbek is.)
• A fordítási egységek tükrözhetnék jobban is az alkalmazás belső logikai felépítését. (Kvázi lehetnek jobban OOP-s jellegű.)

Összességében egyébként a program kód szintjén tetszetős.

ACUEBO

• A README.md-be írni kellene a féléves feladattal kapcsolatban. (Hogy ha két fájlt is használna, akkor mind a kettőt bővíteni kellene.
• A commit üzenetek szövegezése nem megfelelő.

ADVLIC

• A feladat leírása részletes, de az nem egészen derül ki belőle, hogy mi is lenne a programmal a cél.
• A repository aránytalanul nagy méretű (148 MB). A forráskódnak és a dokumentációnak kellene benne lennie. Az asset-eket külön letölthető formában érdemes közreadni.
• A programnak érdemes lehet egy külön jegyzéket készíteni. (Elvileg a félév során elkészített többi feladatnak is ebben kellene lennie.)
• A scene.c túlságosan hosszú. Az alkalmazás kódját az OOP elveknek megfelelően kellene szervezni.
• A Scene struktúra így szintén aránytalanul nagynak tűnik.
• A selected_object_id-hoz lehetne használni enum-ot.

AVVD94

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

BUM37G

A repository-t erősen rendbe kellene szedni. * Hiányzik a feladat specifikációja. * A dll fájlok feltöltése nem szükséges. * A program egyetlen, aránytalanul hosszú (1699-es soros) main.c állományból áll. * Az asset-ek feltöltése nem lett volna indokolt. * include és src jegyzékeket is érdemes lenne használni.

D9J34I

• A repository mérete aránytalanul nagy (199 MB). Sem a build-elt fájlok, sem az asset-ek feltöltése nem indokolt. Érdemes új repository-t létrehozni vagy úgy törölni, hogy a méret clone-ozás után csökkenjen.
• A CMake használata dícsérendő.
• Az elkészítendő feladathoz nem tartozik leírás.
• Vannak a kódban biztató dolgok, de úgy általában rendbe kellene szedni a repository-t.

D9RKIN

• A README.md-ben nagyon szép, érdekes és olvasmányos angol nyelvű leírás található. Az viszont sajnos nem derül ki belőle, hogy mi is lenne ez, ami készül.
• A bináris fájlokat nem kellett volna feltölteni.
• A JSON formátum használata nagyon jó ötlet, dícsérendő.
• A program build-eléséhez szükség lenne egy GNU Make vagy egy CMake leíróra.
• A programkódban vannak tetszetős dolgok, de úgy általában rendbe kellene szedni.

DC1P40

• A feladat leírása hiányzik.
• A bináris fájlokat (dll, exe) nem kellett volna feltölteni a repository-ba. (A repository mérete emiatt aránytalanul nagy.)

DWCPH5

• A gyakorlatok feladatait szépen feltöltötte.
• Néhány exe fájl véletlenül feltöltésre került velük.
• A feladat leírása még hiányzik.

DY98R5

• A feladat leírása alapvetően rendben van. (Mint specifikációba, esetleg lehet írni több konkrétumot, akár tervet is.)
• A forráskód felépítése lehetne jobban OOP jellegű.
• Az ext-hez tartozó relatív include-okat nem tartom indokoltnak.
• A scene.c túlságosan hosszú (1147 soros). Érdemes lenne refaktorálni, kiszervezni több fájlba a tartalmát.
• A main.c szintén túlságosan hosszú (841 soros).

E24HDO

• A repository mérete túlságosan nagy (287 MB). Újat kellene létrehozni, vagy úgy törölni belőle, hogy clone-ozás után kisebb legyen.
• Az asset-eket nem ilyen formában kellene hozzáférhetővé tenni.
• A CMake használata mindenképpen dícsérendő.
• Érdemes lehet külön választani a fejlesztői és a felhasználói leírást.
• Látszik, hogy sok munka van benne. A forráskód viszont elég sok helyen rendezetlen. Alaposabb refaktorálást igényel.

ECY3RI

• A repository-ban lévő dolgok még elég kezdetleges állapotban vannak.

EFM6KW

• GitHub jogosultság probléma. A repository-t nem sikerült clone-oznom.

EJDKWL

• A feladat leírása így nem megfelelő. Nem derül ki belőle, hogy konkrétan mi is készül.
• A CMake használata mindenképpen dícsérendő.
• A repository mérete nem vészesen nagy, de az asset-eket és a build-eket nem kellett volna feltölteni.
• A main.c túlságosan hosszú (617 soros).
• A fordítási egységek kialakítása nem tükrözi a program logikai belső szerkezetét.

EW9LOS

• A repository majdhogynem üres.

FG6XJF

• GitHub jogosultság probléma. A repository-t nem sikerült clone-oznom.

FNJ1Q7

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

GC9GXF

• A repository-ba éppen csak néhány C forrásfájl került feltöltésre.

GEG8Y5

• A leírásban érdemes lenne az elején leírni, hogy milyen feladat készül.
• A main.c aránytalanul nagy (433 soros).
• A forráskód szervezése tükrözhetné jobban az alkalmazás belső logikai elemeit.

H5BEU4

• A feladat specifikációjához a leírás kiindulásnak jó, de így nem elég specifikus, részletes.
• Az exe fájlokat nem szükséges feltölteni a repository-ba.
• Az asset-eket nem ilyen formában kellett volna hozzáadni a programhoz.
• Érdemes lehet legalább egy src jegyzéket használni. (Az include használata is indokolt lehet.)
• A függvények elnevezési módja elég változatos.

HAJWAJ

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

HUYZRA

• A repository aktuálisan gyakorlatilag üres.

I89JMB

• A feladat leírása így elég rövid és sablonos. Jobban, specifikusabban kellene részletezni, hogy mi is lenne ez a program.
• A programkód felépítése nem tükrözi annak belső logikai elemeit.
• A kamera mozgatásához hozzáférhető elegánsabb megoldás is.
• A sorközi kommentekre nem kellene, hogy szükség legyen (vagy csak igen indokolt esetben).

IKISNN

• A feladat leírása gyakorlatilag hiányzik.
• Maga a programkód is szintén elég kezdetleges állapotban van.

ISXBF3

• Az asset-ek és ext jegyzék feltöltése nem tűnik indokoltnak.
• A CSV fájl használata az adatok kezeléséhez mindenképpen jó ötlet, dícsérendő.
• A scene.c aránytalanul hosszú (1445 soros). Refaktorálni kellene a kódot.
• Aránytalanul sok a sorközi komment.

A kódban összességében vannak biztató, tetszetős dolgok, de rendbe kellene szedni.

JDMZZK

• A repository mérete túlságosan nagy (58 MB). (Feltételezhetően valami korábban törlésre került belőle úgy, hogy a git history-ban megmaradt.) Érdemes lehet megpróbálni kiszedni.
• A feladat leírása túlságosan általános, címszavas. Konkrétabban specifikálni kellene, hogy mi hogy fog majd működni, történni a programban.
• Az App struktúra túlságosan sok elemet tartalmaz. Vélhetően ezzel kapcsolatos probléma lehet, hogy a planet_scene.c így túlságosan hosszú (3048 soros).
• A GLU függvénykönyvtárat nem kellene használni.
• Dícsérendő, hogy az asset-eket szépen külön szervezte.

JEH60C

• A feladat megválasztása kifejezetten érdekes, ötletes.
• A program úgy tűnik, hogy még kezdeti fejlesztési fázisban van.

JH3V7T

• Hiányzik a féléves feladat leírása és úgy általában az ahhoz tartozó dolgok.

JSBPC1

• A feladat leírása túlságosan általános. Jobban ki kellene térni az elkészítendő feladat specifikumaira.
• A forráskódok úgy általában hiányoznak a repository-ból.

LBP8ST

Éppen csak egy közel üres readme került feltöltésre. A gyakorlatokon való részvétel minimális volt.

LRP77J

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

M568EG

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

MO0VEY

• Rengeteg felesleges dolog feltöltésre került. Emiatt a repository mérete aránytalanul nagy (408 MB). Újat kellene létrehozni.
• A feladat leírása túlságosan általános, sablonos.
• A programkód felépítése nem tükrözi a program belső logikai szerkezetét.
• Nem kellene GLU-t használni a program elkészítéséhez.
• A pragma-k használata alapvetően nem gond, de helyenként indokolatlannak tűnik.

NHJYQC

• A feladathoz nem találtam specifikációt.
• A program szerkezete elég zavaros. Sem az asset-eket, sem a dll fájlokat nem kellett volna feltölteni.
• A GLFW használatát nem feltétlenül tartom rossz ötletnek, de nem ilyen formában.
• Az extern kulcsszavak használatát úgy általában kerülni kellene.
• A main.c fájl aránytalanul hosszú (491 soros), és úgy általában problémák vannak a program szerkezetével.

NORS38

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

OOKK19

• A választott feladat mindenképpen nagyon jó, eredeti, ötletes.
• A programkód aktuálisan még elég kezdetleges állapotban van.

OREG3H

• A repository mérete aránytalanul nagy (67 MB). Újat kellene létrehozni (vagy legalábbis csökkenteni valahogy).
• A dll fájlok, és úgy általában a binárisok, third party library-k feltöltése nem szükséges.
• Hiányzik a féléves feladat leírása.
• A FreeGLUT és a SOIL függvénykönyvtárakat nem kellene használni.
• Az asset-eket nem kellene a forráskódokkal együtt feltölteni.
• Az extern kulcsszó használatát kerülni kellene.
• A maze.c fájl aránytalanul hosszú (1503 soros). (Úgy általában nem tűnik túl arányosnak a kódbázis.)

PYTZDH

A körlevélben kiküldött űrlapot nem töltötte ki.

QTMKZZ

• A feladat leírása kiindulásnak jó, de lehetne benne több részlet a kivitelezendő elemekre, megoldásokra vonatkozóan.
• A programkódot jegyzékekbe kellene szervezni.
• Nem GLUT-ot kellene használni a program fejlesztéséhez.

R1LM2K

• Szóban említette, hogy milyen alkalmazás fejlesztésében gondolkozik, de érdemes lenne specifikálni.
• Az exe fájlokat nem kellene feltölteni a repository-ba.

R8H3FT

• A féléves feladat specifikációja még hiányzik.
• Aránylag kevés dolog került feltöltésre a félévben elkészített feladatok közül.

RWATOC

• A féléves feladat specifikációja még hiányzik.

S5DP1X

• A README.md még gyakorlatilag üres.
• A félévben elkészített feladatok szépen, rendszerezetten feltöltésre kerültek.

SJGMUZ

• A program ötletes és a kivitelezés is kifejezetten szép.
• A program szerkezete csak hellyel-közzel tükrözi az elkészített elemeket.

Alapos refaktorálást igényel, de egyébként rendben lesz.

SNEK28

• A féléves feladat leírása hiányzik.
• A shader-ek és a CMake használata mindenképpen dícsérendő.
• A saját fejlesztésű és a külső kódok kissé keverednek.
• A main.c lehetne rövidebb.

Rengeteg szép dolog van a programban. Előremutató az újabb technológiák használata. Helyenként viszont aktuálisan még kicsit rendezetlen a kód.

TB1EY5

• A megoldandó probléma megfogalmazása így eléggé nagyvonalú.
• A third party library-k ilyesfajta feltöltése nem indokolt. (Elegendő csak függőségként megadni.)
• Helyenként maradtak a kódban üres fájlok.
• Az új technológiák használata mindenképpen jó ötlet.

Látszik, hogy rengeteg munka volt benne. Kissé rendezetlen érzete van még a kódnak. (Hogy ha engine-ről van szó, akkor a tervezés és a dokumentáció szintén segíthet ezen.)

TKJWDY

• A féléves feladat kiírása és aktuális változatának kódja jelenleg hiányzik.

TL3Y2F

• A repository-ban éppen csak egy közel üres readme van, és egy main.c fájl.

UZZ49L

• A féléves feladat kiírása és aktuális változatának kódja jelenleg hiányzik.

V204JT

• A repository-ban gyakorlatilag csak egy közel üres readme fájl van.

V2WV3W

• A repository-ba a bináris fájlokat és az asset-eket nem szükséges feltölteni.
• A feladat leírása egyáltalán nem rossz, de néhány technikai részletre érdemes lehet jobban kitérni (például a procedurális generálás módjára).
• A forrásfájlok elrendezése nem egészen szokványos, de logikus, következetes, áttekinthető.
• Az interfész leírásánál a kommenteket elegánsabb lehet angolul megadni.
• A main.c-ben a pi konstans értéke gyanús, hogy közel sem annyi.

V64E5R

• A repository mérete indokolatlanul nagy (92 MB). Csökkenteni kellene.
• A gyakorlatokhoz tartozó feladatok szépen feltöltésre kerültek.
• A féléves feladathoz még hiányzik a specifikáció, és úgy általában az ahhoz kapcsolódó dolgok.

V66AJW

• A gyakorlatokhoz tartozó feladatok szépen, módszeresen feltöltésre kerültek.
• A féléves feladat leírása és a forráskódja aktuálisan még hiányzik.

V9ZBUS

• A gyakorlatokhoz tartozó feladatok feltöltésre kerültek. (Véletlenül néhány bináris is köztük maradt.)
• A féléves feladat leírása és a forráskódja aktuálisan még hiányzik.

VTJ4ES

• A repository mérete aránytalanul nagy (383 MB).
• A C SDK-t nem kellene feltölteni.
• A féléves feladat leírása és a forráskódja aktuálisan még hiányzik.

W0E0RY

• A féléves feladatnak aktuálisan még csak a leírása került fel a repository-ba.
• A leírásban jobban ki lehetne térni arra, hogy konkrétan mi és hogy kerül majd megvalósításra. (A “realisztikus hatás”, “fények és árnyékok” és az “élethű megjelenítés” így elég nagyvonalú megfogalmazás.)

WDXVCI

Az első gyakorlaton jelent csak meg. A körlevélben kiküldött űrlapot nem töltötte ki.

WY9VBO

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

XBG3S6

• A feladat kiválasztása érdekes, ötletes. A leírás is szép részletes hozzá.
• A feltöltött forráskód aktuálisan még elég kezdetleges állapotban van.

XVUF92

A félév során eddig egy gyakorlaton jelent meg. A feltöltött repository üres.

Y5KZ61

• A féléves feladat leírása hiányzik.
• A licensz megadása mindenképpen dícsérendő.
• A programhoz egyetlen main.c tartozik. Feltételezhetően kezdeti stádiumban van a fejlesztés.
• A modern eszközök használata mindenképpen tetszetős dolog. A third party library-ket viszont nem szükséges ilyen formában feltölteni (elég csak függőségként jelölni).

Aktuálisan még nem látszik, hogy mi is akarna ez lenni pontosan.

Y8DGYZ

• A gyakorlatokhoz tartozó feladatok egy része szépen feltöltésre került.
• A README.md aktuálisan gyakorlatilag üres.

YE6BLB

• A feladatválasztás érdekes és ötletes, de jelenleg még nincs kellő részletességgel specifikálva.

YNBG2A

• A gyakorlatokhoz tartozó feladatok szépen feltöltésre kerültek.
• A féléves feladat leírása aktuálisan még hiányzik.

Z3VP2R

Gyakorlatokra nem járt, a körlevélben kiküldött űrlapot nem töltötte ki.

ZL3ACR

• A repository mérete aránytalanul nagy. Csökkenteni kellene.
• A third party library-ket nem kellene feltölteni, csak függőségként jelölni!
• A feladat leírásának részletesebbnek kellene lennie.
• A CMake használata mindenképpen előremutató dolog.
• A main.cpp alapján a program C++ fog majd készülni.
• A program forráskódját nyilván majd modulárisan kell szervezni.

ZVPGTZ

• A féléves feladat leírása így eléggé vázlatos.
• A bináris fájlokat nem kellene feltölteni a forráskódok közé.
• Az src/main.c vélhetően még csak egy próbálkozás jellegű kódrész.