Open katarinabrilej opened 3 years ago
Hejla, hvala za popravke, prva si, ki je šla čez mojo kodo hvala za to. Jaz sem končal. Bi lahko preverila ali rabim še kaj dodat? Kako se zažene projekt R na spletni strani? Odprel sem R studio pa ga na internetu odprel, a to je to?
Ko ga zaženem na internetu se mi pod naslovom pojavi še R studio slikica, ki se mi na računalniku, ne nevem, ali je to napaka?
Povezava za RStudio se namenoma prikaže samo pri poganjanju v Binderju - tako je mogoče direktno skočiti v RStudio (brez ponovnega poganjanja slike) za potrebe testiranja in odpravljanja napak.
Opazila sem, da imaš na githubu v sklopu projekta datoteko vaje_9_12.R, ki ne sodi zraven, tako da bi jo bilo bolje umakniti.
Glede uvoza:
Tabela Sloveski_podatki_urejeni
vsebuje tudi podatke, ki so v tabeli Slovenske_regije
, tako da slednje ne potrebuješ. Prav tako je tabela obcine
odveč. V imenih stolpcev naj ne bo šumnikov, torej Povrsina
namesto Površina
. Potrebuješ torej le
Slovenski_podatki_urejeni <- uvozi.regije() %>% group_by(Regija) %>% summarise(Povrsina = sum(povrsina), Prebivalci = sum(prebivalci), Naselja=sum(naselja), Gostota = Prebivalci/Povrsina)
Slovenski_podatki_urejeni$Regija <- gsub('Notranjsko-kraška', 'Primorsko-notranjska', Sloveski_podatki_urejeni$Regija)
Glede vizualizacije:
Pri grafu graf_SLO_Kolicina_proizvodov_na_regio
manjka polovica oznake x osi, torej Vrsta pridelka
namesto pridelka
.
V poročilu se ne vidi celoten naslov grafov, zato jih ali skrajšaj ali pa naslov razdeli v dve vrstici.
Zemljevid najbrž potrebuješ samo en, torej ta, kjer imaš količino proizvoda relativno na število naselij.
Osi grafov naj bodo konsistentno označene, torej recimo vse z veliko začetnico in s šumniki, enako naslovi legend. Vsi grafi naj imajo naslove.
Napredno analizo imaš zaenkrat še v vizualizacija.r
, lahko jo premakneš v analiza.r
in iz tam pobrišeš kar je ostalo od vzorčnega repozitorija.
V graf_indeks
se težko razloči toliko podatkov, saj pride kar veliko črt. Morda bi bilo bolje to narediti v Shiny-ju. Torej, da bi bilo možno izbrati pridelek in pa tudi občino (oz. več občin).
Tudi za graf graf_indeks_reduciran
je to morda bolj smiselno narediti v Shyniju.
Mi pa ni čisto jasno kaj naj bi predstavljal graf, ki je trenutno v Shinyju.
Pa tudi še kar nekaj slovničnih napak je za popraviti v poročilu.
@BlackPhoenixSlo, glede na to, da ti projekt deluje, bom dodal povezavo na repozitorij za zagovore, tako da se lahko z @alenFMF domeniš za termin zagovora. Če boš do takrat še kaj spreminjal, mi javi, da posodobim povezavo.
Poleg komentarjev, ki jih je podala @katarinabrilej, bi opozoril še na naslednje:
element_text
poskusi dodati parametra hjust=1
in vjust=0.5
(oba vzameta vrednosti med 0 in 1).graf_indeks_reduciran
bi dodatno opozoril na to, da vsaka črta prikazuje podatke na različni lestvici, zato je tak graf nejasen in zavajajoč - pravzaprav sta črti za surove in "reducirane" podatke le večkratnik ena druge, tako da nima smisla prikazovati obeh na istem grafu (sploh glede na to, da hkrati prikazuješ eno samo regijo). Namesto tega predlagam, da narediš točkovni graf, kjer imaš število naselij na osi x in količino proizvodov na osi y, točke pa pobarvaš glede na regijo, vse skupaj pa lahko animiraš po letih (ali pa kaj podobnega).Še to: za poganjanje poročila v Binderju imaš v README.md
povezavo, kjer zraven piše Shiny.
Živjo, jaz se popravil napake, katere sta mi zgoraj napisala (razen vseh slovničnih napak). Bi prosim posodobil link v repozitoriju?
Sem posodobil.
Pri uvozu zemljevida je potrebno nastaviti
encoding = UTF-8
, torejTako ne bo potrebno popravljati imen regij z izjemo Spodnjeposavske regije.
Oznake regij dodaš na zemljevid na sledeč način: