Projekt

[IanLampic]

Sporočam, da oddajam projekt.

[katarinabrilej]

Hvala, bomo ocenili.

[M4rble]

Poročilo se ne prevede. Povsod kjer v projektu uporabljaš funkciji source ali read, odstrani path svojega računalnika pusti le lokalni path znotraj projekta (torej "/Users/ianlampic/Desktop/APPR-2021-22/analiza/analiza.r" spremeni v "analiza/analiza.r"). Tako bo koda generično delovala na kateremkoli računalniku. Nato bi se moralo poročilo ustrezno prevesti.

[M4rble]

Ko popraviš po zgornjem komentarju, projekt zadošča za pozitivno oceno. Vseeno nekaj komentarjev za možne izboljšave:

• tvoji grafi v vizualizaciji so zelo nepregledni, nekateri tudi nesmiselni (kar se splača popraviti, saj je to nekako ključni del tega predmeta)
• 1. graf - ni zares smiseln v taki obliki, saj zelo težko primerjamo recimo število obolelih za določeno bolezen v letu 2011 z letom 2019. Precej bolje bi bilo, če imel na x-osi leta in nato število obolelih v vsakem letu prikazal s točkami/jih povezal. Tako bi lahko veliko lažje primerjal tako število obolelih po letih, kot število obolelih za različne bolezni. Če za kakšno bolezen ni podatkov, ali jih je premalo, jo lahko na grafu seveda izpustiš, je pa priporočljivo to potem komentirati v poročilu.
• 2. graf - možne izboljšave: odstranitev poimenovanja osi (v tortnih diagramih je običajno iz naslova in legende precej jasno kaj prikazuje), sprememba imena legende v recimo Bolezni, odstranitev belih črt okoli tortnega diagrama (theme_void() ), dodajanje odstotkov na krožne izseke (težko je razbrati, kateri je večji, številke bi pri tem pomagale)
• 3. graf - zelo nepregleden. Lahko bi izbral nekaj držav (recimo 5) z največjim številom bolezni in jih primerjal med seboj. Lahko bi tudi prikazoval samo najbolj pogoste bolezni, ne vseh, tako bi bil graf še bolj pregleden. Zelo dobro je da si uporabil facet_wrap, vendar je prikazanih držav absolutno preveč in postane graf nepregleden. Poleg tega v poimenovanju osi in naslovih uporabljaj šumnike. Tudi legenda na takem grafu ni preveč smiselna, vsaj ne v te obliki
• 4. graf - ponovno nepregleden. Bolje bi bilo, če bi samo seštel število ljudi, ki so zboleli za posamezno boleznijo v EU, saj delitev po državah tu nima neke dodane vrednosti. Naprej spet uporaba šumnikov v oznakah oseh, naslovu, legendi, ...
• 5. graf - tu je taka oblika grafa precej bolj primerna kot prej, saj je graf tudi če vključiš vse države še vedno pregleden. Leta na x-osi se ponavadi označuje s celo vrednostjo, ne z 2017.5 (saj to nima zares smisla). Spet uporaba šumnikov.
• 6. graf - primeren, isto kot pri 5., leta in šumniki
• 7. graf - šumniki
• 8. graf - dobra ideja, ponovno nepregleden, zdi se kot da so vsi pravokotniki enako visoki. Predlagal bi, da si spet izbereš nekaj držav (morda iste kot prej - lahko na začetku poročila poveš, da se boš zaradi preglednosti osredotočal le na Slovenijo + nekaj držav) in prikažeš le podatke za njih, na večje, tako da je jasno ali je v Avstriji v letu 2015 povprečna starost 55 ali 58 ali kaj drugega. Poleg tega bi bilo smiselno komentirati, zakaj si si kot mejo izbral ravno 62 let (omeniš da je povprečje, česa?). Spet šumniki.
• 9. graf - spet dobra ideja, spet bi bilo smiselno morda ločiti Islandijo in Luksemburg od ostalih - tako bi lahko lažje primerjal razmerja ostalih držav.

Poleg tega bi se dalo tudi

• izboljšati napovedni model - primerjati več različnih modelov s prečnim preverjanjem in nekje komentirati rezultate
• izboljšati Shiny aplikacijo (izbrati kakšne druge grafe, ki so malo bolj raznoliki med seboj, da bi bile spremembe v aplikaciji bolj očitne)
• natančneje komentirati grafe in delo (bolj podrobno graf predstavlja, kaj lahko iz njega razberemo/izvemo v širšem smislu), ter v poročilo dodati zaključek.

[IanLampic]

Pozdravljeni, sporočam, da sem naredil popravke in spremenil path na lokalni path.

Lep pozdrav, Ian Lampič

V V čet., 25. avg. 2022 ob 01:24 je oseba M4rble @.***> napisala:

Ko popraviš po zgornjem komentarju, projekt zadošča za pozitivno oceno. Vseeno nekaj komentarjev za možne izboljšave:

• tvoji grafi v vizualizaciji so zelo nepregledni, nekateri tudi nesmiselni (kar se splača popraviti, saj je to nekako ključni del tega predmeta)
• 1. graf - ni zares smiseln v taki obliki, saj zelo težko primerjamo recimo število obolelih za določeno bolezen v letu 2011 z letom 2019. Precej bolje bi bilo, če imel na x-osi leta in nato število obolelih v vsakem letu prikazal s točkami/jih povezal. Tako bi lahko veliko lažje primerjal tako število obolelih po letih, kot število obolelih za različne bolezni. Če za kakšno bolezen ni podatkov, ali jih je premalo, jo lahko na grafu seveda izpustiš, je pa priporočljivo to potem komentirati v poročilu.
• 1. graf - možne izboljšave: odstranitev poimenovanja osi (v tortnih diagramih je običajno iz naslova in legende precej jasno kaj prikazuje), sprememba imena legende v recimo Bolezni, odstranitev belih črt okoli tortnega diagrama (theme_void() ), dodajanje odstotkov na krožne izseke (težko je razbrati, kateri je večji, številke bi pri tem pomagale)
• 1. graf - zelo nepregleden. Lahko bi izbral nekaj držav (recimo 5) z največjim številom bolezni in jih primerjal med seboj. Lahko bi tudi prikazoval samo najbolj pogoste bolezni, ne vseh, tako bi bil graf še bolj pregleden. Zelo dobro je da si uporabil facet_wrap, vendar je prikazanih držav absolutno preveč in postane graf nepregleden. Poleg tega v poimenovanju osi in naslovih uporabljaj šumnike. Tudi legenda na takem grafu ni preveč smiselna, vsaj ne v te obliki
• 1. graf - ponovno nepregleden. Bolje bi bilo, če bi samo seštel število ljudi, ki so zboleli za posamezno boleznijo v EU, saj delitev po državah tu nima neke dodane vrednosti. Naprej spet uporaba šumnikov v oznakah oseh, naslovu, legendi, ...
• 1. graf - tu je taka oblika grafa precej bolj primerna kot prej, saj je graf tudi če vključiš vse države še vedno pregleden. Leta na x-osi se ponavadi označuje s celo vrednostjo, ne z 2017.5 (saj to nima zares smisla). Spet uporaba šumnikov.
• 1. graf - primeren, isto kot pri 5., leta in šumniki
• 1. graf - šumniki
• 1. graf - dobra ideja, ponovno nepregleden, zdi se kot da so vsi pravokotniki enako visoki. Predlagal bi, da si spet izbereš nekaj držav (morda iste kot prej - lahko na začetku poročila poveš, da se boš zaradi preglednosti osredotočal le na Slovenijo + nekaj držav) in prikažeš le podatke za njih, na večje, tako da je jasno ali je v Avstriji v letu 2015 povprečna starost 55 ali 58 ali kaj drugega. Poleg tega bi bilo smiselno komentirati, zakaj si si kot mejo izbral ravno 62 let (omeniš da je povprečje, česa?). Spet šumniki.
• 1. graf - spet dobra ideja, spet bi bilo smiselno morda ločiti Islandijo in Luksemburg od ostalih - tako bi lahko lažje primerjal razmerja ostalih držav.

Poleg tega bi se dalo tudi

• izboljšati napovedni model - primerjati več različnih modelov s prečnim preverjanjem in nekje komentirati rezultate
• izboljšati Shiny aplikacijo (izbrati kakšne druge grafe, ki so malo bolj raznoliki med seboj, da bi bile spremembe v aplikaciji bolj očitne)
• natančneje komentirati grafe in delo (bolj podrobno graf predstavlja, kaj lahko iz njega razberemo/izvemo v širšem smislu), ter v poročilo dodati zaključek.

— Reply to this email directly, view it on GitHub https://github.com/IanLampic/APPR-2021-22/issues/1#issuecomment-1226593233, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATYI5CEAQMKJWNTCHUZOTA3V22VJ5ANCNFSM57QUTIYA . You are receiving this because you authored the thread.Message ID: @.***>

[katarinabrilej]

Sem vnesla oceno v spletno učilnico.

Lep pozdrav, Katarina