Grafika wchodzi bez review, bo jej obejrzenie za dużo kosztuje
Narzędzia do code review pokazują zmieniony .uasset jako „plik binarny, 9,5 MB”. Żeby zobaczyć, co grafik naprawdę zrobił, reviewer musi zsynchronizować zmianę, otworzyć edytor, znaleźć asset i porównać go z tym, co pamięta z zeszłego tygodnia. Więc większość zespołów tego nie robi. Touch Review robi to porównanie za Ciebie, w tym samym miejscu, w którym przeglądasz kod.
Widzisz node, który się zmienił, a nie binarny plik
Obie wersje grafu stoją obok siebie i ruszają się razem: przesuwasz albo przybliżasz jedną, druga idzie za nią. Lista zmian po prawej mówi zwykłymi słowami, co się stało. Klikasz pozycję i obie strony centrują się na tym nodzie.
- Każdy graf w Blueprincie: event graph, funkcje, makra, do tego zmienne, komponenty i class defaults.
- Komentarz na nodzie. Trzyma się nodu, a nie numeru linii.
Grafy materiałów dokładnie takie, jak w edytorze
Graf eksportuje sam Unreal Engine, więc każdy node, kolor i podgląd zgadza się z tym, co widział grafik. Texture Sample pokazują prawdziwe tekstury, a stara strona pokazuje stare.
- Dodane, usunięte, przepięte, przestrojone: zmieniona stała ma na liście starą i nową wartość.
- Material Instance pokazują, które parametry nadpisano i jak.
Która część modelu naprawdę się ruszyła?
Otwierasz zmieniony mesh i oglądasz obie wersje razem, z teksturami. Przełączasz na mapę cieplną i odpowiedź jest w kolorze: zielone się nie ruszyło, czerwone tak. Słup tej latarni jest nietknięty, klosz urósł.
- Sześć widoków: obok siebie z jedną kamerą, nałożenie, mapa cieplna, siatka, UV checker i glina.
- Pinezki na modelu. Komentarz przypięty do punktu na meshu.
Łatka widoczna bez mrużenia oczu
Postaw obie wersje obok siebie, przesuwaj suwak między nimi, przenikaj jedną w drugą albo pokaż samą różnicę. Touch Review liczy zmienione piksele, więc „przemalowano 9% muru” to fakt na ekranie.
- Rysuj po niej. Zakreśl miejsce kółkiem, strzałką albo ramką i dołącz do niego komentarz.
- Eksportuj oznaczony obraz jako PNG dla grafika.
I code review, które już znasz
Diffy obok siebie albo w jednej kolumnie, z kolorowaniem składni, komentarze do linii, wątki, które się zamyka. Każdy komentarz ma wagę, więc autor wie, co musi poprawić przed akceptacją, a co jest tylko luźną uwagą.
- Blocker, sugestia, drobiazg, pochwała: blocker wstrzymuje akceptację, dopóki nie zostanie rozwiązany.
- Szablony komentarzy na rzeczy, które piszesz co tydzień.
