Tivoli: Peer review gui - [merged]

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 12:16

_Merges Peer_reviewGUI -> master

Tivoli-projektin jsrunner / taulukot yms vertaisarvioinnin seuraamisen kokeilemista varten.

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 12:18

@alexander.sodergard Katson tämän vähän myöhemmin läpi. Oliko tästä jotain testidokumenttia pystyssä jotta voisi kokeilla jo ennen mergeä?

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 12:25

@alexander.sodergard niin ja sitten joku ohje (joku pari lausetta mistä valikosta löytyy tai mitä pitää pohjasta muuttaa yms) siitä miten toimintoja käytetään.

[dezhidki]

added 1 commit

• 634b8994 - Triger CI

Compare with previous version

[dezhidki]

added 1 commit

• f979424a - Fix formatting

Compare with previous version

[dezhidki]

added 1 commit

• 3aec6b45 - simcirtest: revert formatting

Compare with previous version

[dezhidki]

added 2 commits

• 490a8df5 - JSRunner: clean up workspace
• 3b54bfba - timApp: revert package-lock.json

Compare with previous version

[dezhidki]

added 1 commit

• cc1e823c - annotations: remove reduntant options

Compare with previous version

[dezhidki]

added 5 commits

• d4985724 - Fix formatting
• fc8408b0 - simcirtest: revert formatting
• 8d6bab88 - JSRunner: clean up workspace
• 27ee25cf - timApp: revert package-lock.json
• d8588018 - annotations: remove reduntant options

Compare with previous version

[dezhidki]

added 3 commits

• c4e152e4 - jsrunner: update gitignore
• 539184a5 - tools: Fix linter issues
• fbde6ec7 - Remove peer_review.component.ts

Compare with previous version

[dezhidki]

added 3 commits

• fbde6ec7...09c5f775 - 2 commits from branch tim-jyu:master
• e60f5feb - Merge branch 'master' into Peer_review_GUI

Compare with previous version

[dezhidki]

added 1 commit

• fbde1ad3 - Clean up before merge

Compare with previous version

[dezhidki]

Tein tähän väliin muutoksia, jotta saadaan nopeammin MR läpi:

• Formatoin kaikki tiedostot black . sekä npmi formatfix
• Korjasin joitain linttereiden ilmoittamia ongelmia
• Lisäsin gitignoren jsrunner/server-kansiolle (se kai yritettiin editoida VS:llä tai VSCode:lla, mikä
• Poistin peerreview-pluginin ja vastaavan komponentin, koska se ei ole tällä hetkellä käytössä missään
• Korjasin packages-lock.json takaisin. Tarkista lokaalissa koneessa, että sinulla on NPM versio 6 asennettuna! Nyt siis lock-tiedosto oli väärin.

Laitan nyt approve ja mergeän tuotantoon.

[dezhidki]

approved this merge request

[dezhidki]

mentioned in commit 245a355fa6917be4f8febde831f5fe5fe69c2b8e

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 15:28

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1329

Mitenkähän tässä käy jos on aiempia vastauksia, niissä velppejä, sitten palautettu uusi vastaus ja velpattu sitä uutta vastausta? Ne vanhatkin velpit taitaa tulla mukaan laskentaan. Muut jsrunnerien kenttien arvot otetaan aina uusimmasta vastauksesta. Tuolla ylempänä on get_annotations_with_comments_in_document, jossa haetaan kaikki velpit, eli tässä taidetaan käsitellä myös edellisen vastauksen velppejä.

Prioriteetiltaan ei varmaan kovin tärkeä, ehkä voinee olettaa että vertaisarvioinnin kohdalla vastaaminen on suljettu ja velppejä tulee vain viimeiseen vastaukseen. Mutta jatkokehitysohjeisiin kannattaa merkitä maininta siitä että velpit pitäisi ehkä olla vain viimeisestä vastauksesta (tai antaa joku tapa viitata tietyn vastauksen velppeihin)

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 15:29

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1331

En muista ulkoa onko tools.ts:ssä jo funktio jolla parsitaan taskID:tä tai kaivetaan sieltä ulos pelkkä tehtävän nimi. Jos ei, niin kannattaa saman tien tehdä semmoinen ja käyttää sitä tässä.

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 15:32

Commented on timApp/modules/jsrunner/server/routes/answer.ts line 51

Tälle tulee varmaan myöhemmin joku vähemmän "test"-nimi

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 15:40

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1325

Ennen projektin päättymistä laittakaa kaikkiin uusiin funktioihin joku ihan palikkatason jsdoc-kommentti (pari sanaa mitä funktio/parametrit tekee ja mitä palautetaan). Pycharm luo pohjan jos ts-tiedostossa funktiota edeltävälle riville kirjoittaa /** ja painaa enter.

Vaikka koodi olisi kuinka selkeää ja "itsestään kommentoituvaa", ja tuntuu että se aukeaa kenelle tahansa kuka koodia lukee ilman mitään kommentointia, niin parin vuoden päästä saattaa tulla tilanne että funktiota on refaktoroitu ja joku uusi projektilainen / työntekijä tms tutustuu koodiin, ja silloin kaikki dokumentaatio nopeuttaa sitä prosessia. (saattaa olla vähän tekopyhää sillä itseltäni jäänyt usein nuo kommentit pois)

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 15:49

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1344

Jos tulee velppejä joilla vain kommentoidaan vastausta, eli pisteenä on 0, ne laskee tuota keskiarvoa. Pitäiskö velps.length sijaan ottaa pistevelppien lukumäärä?

Minusta koko vertaisarvoinnista annettu pistemäärä olisi parempi antaa yksittäiseen kenttään answerbrowserissa (ja tietokannassa se menisi peer_review-tauluun), sitä on paljon selkeämpi ylläpitää kuin pyöritellä pistevelppien summia. Vesa ehdotti sellaista että tuollaiseen kenttään olisi jonkinlainen avustin jolla saa pistevelppien summan suoraan kentän arvoksi. Näitä kannattaa tilaajan kanssa miettiä

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 15:55

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1351

Tämä funktio siis hakee lukumäärään siitä kuinka monta arviointia/velppiä vastaaja on tehnyt? Funktion nimestä mietin ensin että se on kuinka monta velppiä tai arviointia vastaajalle on annettu. Esim tällaisia tilanteita varten tuo "ylimääräinen" dokumentointi on hyvästä.

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 16:00

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1372

Tää taitaa toistua monta kertaa muissa funktioissa. Vois siirtää omaan funktioon ja käyttää sitä

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 16:19

Commented on timApp/modules/svn/templates/video/1/Table And Fields line 111

Tässä tais tulla NaN, kun arvostelija ei antanut yhtään pistevelppiä vaan pelkkiä kommentteja. Se vois varmaan saman tien olla 0? Tai ehkä miettiä tilaajan kanssa kumpi parempi.

https://timdevs01-5.it.jyu.fi/teacher/users/test-user-1/harjoitus1#t2_demotable

[dezhidki]

In GitLab by @sijualle on Apr 8, 2022, 16:29

@alexander.sodergard @eljosilv @marjuk @pamipoyr Laitoin muutaman kommentin, ehkä enemmänkin pohdittavaksi toteutustavoista kuin varsinaisesta koodista. Kannattaa kaikkien ryhmäläisten selata nuo läpi

[dezhidki]

In GitLab by @alexander.sodergard on Apr 8, 2022, 17:03

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1344

"Jos tulee velppejä joilla vain kommentoidaan vastausta, eli pisteenä on 0, ne laskee tuota keskiarvoa. Pitäiskö velps.length sijaan ottaa pistevelppien lukumäärä?"

joo tuon tajusin vasta eilen illalla niin tein sitten nopeesti getPoints-funktion joka palauttaa vaan pistemäärän, jos tässä ekassa oli tarkoitus antaa vain 1 pistevelppi/arvioija?

[dezhidki]

In GitLab by @alexander.sodergard on Apr 8, 2022, 17:07

Commented on timApp/modules/jsrunner/server/routes/answer.ts line 51

Nimi on jäänyt aivan alkuvaiheista kummittelemaan.

[dezhidki]

In GitLab by @alexander.sodergard on Apr 8, 2022, 17:08

Commented on timApp/modules/jsrunner/server/routes/tools.ts line 1351

"Tämä funktio siis hakee lukumäärään siitä kuinka monta arviointia/velppiä vastaaja on tehnyt?" Joo näin on. Lisään tähän ja muihin kommentoinnit