Poslední příspěvky

1

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od Martin Poljak kdy Dnes v 11:52:03 »

Co si vsimam, tak ludia co robia cez LLM si nechavaju aj code review robit cez LLM a po tyzdni sa do kodu uz vobec nepozru (a to nazyvaju sami seba programatormi).

Nevím, u nás děláme skoro jenom pomocí LLM, na code-review máme komplexní skills, ty rutinně používáme, ale stejně děláme code-review dost pečlivě i ručně. Takže to záleží. Není to žádné pravidlo. Vždycky platilo, že jsou lidé lepší a horší a bude to platit i dál. AI to neoslabuje, spíš naopak zesiluje.

..mňa by skôr zaujímalo, či vznikajú aj "názorové konflikty" medzi programátormi [...] Máte aj takú skúsenosť, že CedeReview programátori boli menej skúsení od Vás a prišlo k "ťažkej" výmene názorov ?

Naprosto. Za mě code-review velmi často totálně rozbíjí týmovou morálku. K tomu, aby to tak nebylo je třeba potkat fakt rozumné a neagresivní lidi. A takových je bohužel celkem málo.

Takže neřešíte třeba až tak úplně business logiku, jenom jestli tam nejsou nějaký úplně strašný a nebezpečný prasárny?

Záleží, na jaké úrovni myslíte. Na té nejnižší samozřejmě ano nebo pokud je tam nějaký zjevný problém. Na té vyšší ne. Napsat kód je 30 % práce vývojáře. Takže kdybych to měl řešit komplexně, vlastně budu tu práci dělat z velké části podruhé.

U LLM, ale i non-LLM kódu je potřeba hlavně pochopit, jak daný commit řeší zadaný problém. Dělá to, co má? Nedělá to něco navíc, co nebylo v zadání [...]

To zní, jakože na tom review musíte strávit snad stejně času jako původní řešitel, ne?

To má hlavně dělat ten, kdo tu AI vede a zadává ji práci.

Některý commit lze schválit "formálně", když člověk na první dobrou vidí, že je vše v pořádku.

Jak tohle jste schopen posoudit na první dobrou? Nebo to je myšleno spíš v kontextu triviálních změn?

Za mě třeba přesně tak.

2

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od Zopper kdy Dnes v 11:37:11 »

U LLM, ale i non-LLM kódu je potřeba hlavně pochopit, jak daný commit řeší zadaný problém. Dělá to, co má? Nedělá to něco navíc, co nebylo v zadání (pochopil pisatel kódu zadání, bylo jasné, co má dělat)? Nedělá to moc složitě (z výpočetního hlediska, struktury)? Drží to guidelines pro psaní kódu? Je kód srozumitelný, dobře okomentovaný, používá smysluplné identifikátory?

To zní, jakože na tom review musíte strávit snad stejně času jako původní řešitel, ne?

Vůbec ne. U review nemusím vymýšlet, jak to celé poskládat, nemusím si projít těch 10 slepých cestiček, a tak dál. Pokud kód následuje mě známé vzorce, tak typicky nepřemýšlím řádek po řádku, ale po blocích. Podobně jako text nečtu slovo po slově, ale jako větu (dyslektici prominou). Například ohledně zmiňované výpočetní složitosti typicky budu chytat jen vnořené smyčky, častý opakovaný un-boxing při práci s jednoduchými typy, a takové věci. Ve většině případů ale nebudu řešit, jestli tohle je o 10% rychlejší, nebo pomalejší výpočet.

Některý commit lze schválit "formálně", když člověk na první dobrou vidí, že je vše v pořádku.

Jak tohle jste schopen posoudit na první dobrou? Nebo to je myšleno spíš v kontextu triviálních změn?

Za mě tohle jsou triviální změny (ale definuj triviálnost ), nebo reviews, kde já jsem jen pro formální approval ("jo, může se použít tahle naše deprecated věc, protože to je jen přesun funkcionality z jedné třídy do druhé a existuje ticket na odstranění").

V dnešní době by ale každopádně měl první kolo code review dělat nějaký LLM, odhalí toho docela dost a rychle.

JJ, na tohle jsou ty plechovky super. U vygenerovanýho kódu jim pořád úplně nevěřím, ale review je možnost napáchat škody minimální, ale většinou něco odchytí.

+1, a z mojí zkušenosti mají větší šanci si všimnout chybějící změny, než člověk. Například přidal se nový volitelný field, to zahrnuje změnu 5 různých DTO, ale změnily se jen 4. Testy nepadají, protože se to taky přidalo jen do 4 testů z 5.

3

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od B.H.L. kdy Dnes v 11:01:29 »

Procházím to a hledám v tom principiální chyby nebo věci, které by mohly ohrozit výkon aplikace.

Takže neřešíte třeba až tak úplně business logiku, jenom jestli tam nejsou nějaký úplně strašný a nebezpečný prasárny?

Co si vsimam, tak ludia co robia cez LLM si nechavaju aj code review robit cez LLM a po tyzdni sa do kodu uz vobec nepozru (a to nazyvaju sami seba programatormi).

Což je ovšem naprosto pochopitelný - jenom kontrolovat cizí kód je nevděčná práce a troufnu si tvrdit, že přinejlepším vědomí a svědomí málokdo dokáže jenom čtením porozumět cizímu kódu tak dobře, jako by ho psal sám.

Ale u části projektů vlastně na kvalitě kódu nesejde - Hlavně že to dělá, co to dělat má.

U LLM, ale i non-LLM kódu je potřeba hlavně pochopit, jak daný commit řeší zadaný problém. Dělá to, co má? Nedělá to něco navíc, co nebylo v zadání (pochopil pisatel kódu zadání, bylo jasné, co má dělat)? Nedělá to moc složitě (z výpočetního hlediska, struktury)? Drží to guidelines pro psaní kódu? Je kód srozumitelný, dobře okomentovaný, používá smysluplné identifikátory?

To zní, jakože na tom review musíte strávit snad stejně času jako původní řešitel, ne?

Některý commit lze schválit "formálně", když člověk na první dobrou vidí, že je vše v pořádku.

Jak tohle jste schopen posoudit na první dobrou? Nebo to je myšleno spíš v kontextu triviálních změn?

V dnešní době by ale každopádně měl první kolo code review dělat nějaký LLM, odhalí toho docela dost a rychle.

JJ, na tohle jsou ty plechovky super. U vygenerovanýho kódu jim pořád úplně nevěřím, ale review je možnost napáchat škody minimální, ale většinou něco odchytí.

4

Distribuce / Re:Distro pro 13letého kluka na herní PC

« Poslední příspěvek od hlp84939 kdy Dnes v 10:09:16 »

Vzhledem k tomu, v jakem stavu je Windows 11 (a taky aby si ten pocitac za chvili nezaviroval),

A v jakém stavu je Windows 11? Osobně ho mám ve všech svých PC už přes rok a rozdíl oproti W10 jsem prakticky nezaregistroval (až na odlišnou nabídku Start a kontextové menu, což jsou ovšem prkotiny). A zavirovat si PC? To zase úplně triviální není, pokud k tomu uživatel nejde cíleně (vypnutí antiviru, firewallu atd.).

5

Vývoj / Re:Jak děláte code review?

« Poslední příspěvek od HelloWord kdy Dnes v 09:51:07 »

..mňa by skôr zaujímalo, či vznikajú aj "názorové konflikty" medzi programátormi, ktorí CodeReview vykonávajú. V praxi čo sledujem, CodeReview vykonávajú aspoň viac než dvaja programátori. Nebavíme sa o malých commitoch. Vždy ma hlavné slovo jeden z programátorov ? Leader ? Máte aj takú skúsenosť, že CedeReview programátori boli menej skúsení od Vás a prišlo k "ťažkej" výmene názorov ?

6

Distribuce / Re:Neaktualizované balíky v Debianu testing

« Poslední příspěvek od rmrf kdy Dnes v 09:35:10 »

Já na firefox kašlu, používám esr verzi ze stable. Kodi mám na samostatném hardwaru a to se upgraduje samo, pokud má z čeho.

Kromě toho se mrkni sem: https://packages.debian.org/search?keywords=kodi&searchon=names&suite=all&section=all a uvidíš, že kodi v testingu vůbec není, zato ale je v unstable. A tohle je zrovna typický příklad toho, že se některé balíky do testingu ani nemusejí dostat, protože dřív, než z unstable propadnou do testingu, objeví se v unstable nová verze a ta předchozí, čekající na zařazení do testingu, zmizí.

A i tak může stát, že tu novou verzi z testingu/unstable nenainstaluješ, protože je přeložena s jinými závislostmi, které nedokážeš splnit buď z výše uvedeného důvodu a nebo proto, když se to třeba pokoušíš instalovat do stable, že by to vyžadovalo přeinstalaci skoro celého systému - aby se vyřešily ty závislosti. Takže nakonec skončíš s tím, že tam máš z větší části unstable, a i to se občas rozbije.

7

Distribuce / Re:Distro pro 13letého kluka na herní PC

« Poslední příspěvek od Zopper kdy Dnes v 09:19:04 »

To mi připomíná... Ještě bych doplnil otázku: Co ten kamarád, je na stejné vlně, bude motivovat syna, aby hledal cestu bez Windows? Nebo tohle jde úplně mimo něj, tobě ten Linux sice odkýve, ale jak za ním syn příjde s prvním problémem a on uvidí neznámý systém, tak ty Windows bude iniciovat sám a ještě u toho bude nadávat?

8

Distribuce / Re:Neaktualizované balíky v Debianu testing

« Poslední příspěvek od peekaa kdy Dnes v 09:09:35 »

No já na tomhle stroji mám testing už pár let, 7, a tohle se mi stalo poprvé, s posledním testingem. Mám to hlavně kvůli Firefoxu, Kodi - dříve na stable byly hodně staré verze, možná se to již změnilo, ale půl roku je pro mě moc. Jen kanclářské práce plus filmy, seriály.

9

Distribuce / Re:Distro pro 13letého kluka na herní PC

« Poslední příspěvek od hlp84939 kdy Dnes v 09:04:33 »

Jsem upřímně zvědavá, jak dlouho 13 letému klukovi vydrží na herním PC Linux. Podle mě hned ten samý den, jen co bude u PC sám, si spustí nějakou oblíbenou hru, zjistí že něco nefunguje, stáhne si Windows, nainstaluje je a bude pařit bez nejmenších problémů. Akorát se obávám, že stáhne nějaké "cracklé" ISO a bude mít zaděláno na problém. Nestálo by za to místo šíření evangelia mu ty Windows dát prostě hned?

Přesně tak. Nehledě na to, že ten PC tutově nebude používat jenom na hry, ale třeba taky na office (Word, Excel, PowerPoint), takže Windows jsou jasná volba.

10

Distribuce / Re:Distro pro 13letého kluka na herní PC

« Poslední příspěvek od Zopper kdy Dnes v 08:42:43 »

Dřív, než tam něco nainstaluješ, se zeptej, co hraje. Protože i když spousta her už přes Proton/Steam jede krásně, tak jestli hraje třeba Fortnite, tak máš s Linuxem utrum - anticheat ochrany u tohoto typu her v lepším případě nedovolí hru na Linuxu spustit, v horším ho rovnou označí za cheatera a dají ban.

Jo a jestli má/chce třeba gamepad, tak to taky může být zábava. Například rozchodit Xbox controller s jejich 2.4GHz wireless USB donglem vyžaduje kompilaci jaderného modulu, který hledáš někde pod GitHubech, DKMS, a tyhle srandy, a jednou za čas se to rozbije.