Dokončan projekt

[JanKolenc]

Pozdravljeni,

Dokončal sem projekt, pa me zanima če je potrebno še kaj manjšega popraviti.

Jan

[MarvinHerzog]

Projekt se mi prevede pravilno in zgleda super, imam samo par manjših komentarjev:

• Odstavek STOPNJA TVEGANJA REVŠČINE se ti v poročilu pojavi dvakrat.
• Vidim, da si pri uvozu zemljevida popravljal imena regij (Slovenija$Regija <- gsub('GoriÅ¡ka', 'Goriška', Slovenija$Regija)). Če v raznih uvozih naletiš na težave s šumniki ali ostalimi znaki, je ponavadi najbolje, da te težave rešuješ kar pri viru tako, da poskusiš najti prave nastavitve glede encodinga (ter locale če je to potrebno, kot si storil v setup delu z vrstico Sys.setlocale('LC_ALL', 'Slovenian')). V primeru uvoza zemljevida lahko to narediš tako, da v funkcijo uvozi.zemljevid dodaš parameter encoding = "UTF-8" (meni so se imena regij sicer že uvozila s pravilnimi šumniki brez pomoči gsub ali nastavljanja encoding-a, zato nisem 100% glede UTF-8)
• Za v prihodnje ti svetujem, da se izogibaš šumnikom v imenih spremenljivk (npr izobrazba.16.ž), saj zna to na nekaterih sistemih povzročati težave. Šumniki v elementih tabel so ok.
• Ponekod odstranjuješ stolpce z večkratnim zaporednim indeksiranjem, v smislu brezposelnost[,-1][,-3][,-3]. Isto lahko dosežeš tako, da podaš vektor indeksov stolpcev, ki jih želiš odstraniti, tako kot tu: brezposelnost[,-c(1,4,5)]. Alternativno lahko odstranjuješ stolpce po njihovih imenih s pomočjo dplyr-ove funkcije select.
• Za graf graf.prag.tveganja.izobrazba.s si lahko prihraniš precej dela s konstrukcijo data.frame-a izobrazba.16.s in kar uporabiš data frame izobrazba.16. Tu sta dva načina, kako lahko dosežeš podoben rezultat:

S pomočjo funkcije interaction()

af.prag.tveganja.izobrazba.s <- ggplot((data=izobrazba.16), aes(x=Leto, y=Odstotki, col=interaction(Izobrazba,Spol,sep = ", ")))+  
  scale_y_continuous('Odstotki',breaks = seq(0,40, 2), limits = c(0, 35)) + 
  geom_point() + geom_line() + theme_classic() +  scale_x_continuous('Leto',breaks = seq(2008, 2017, 1), limits = c(2008, 2017))

S pomočjo parametra shape

graf.prag.tveganja.izobrazba.s <- ggplot((data=izobrazba.16), aes(x=Leto, y=Odstotki, col=Izobrazba,shape=Spol))+  
  scale_y_continuous('Odstotki',breaks = seq(0,40, 2), limits = c(0, 35)) + 
  geom_point() + geom_line() + theme_classic() +  scale_x_continuous('Leto',breaks = seq(2008, 2017, 1), limits = c(2008, 2017))

Kot rečeno so pa to bolj ali manj malenkosti, projekt se mi zdi dobro narejen in je pripravljen za oddajo. Ko ga boš želel oddati kar taggaj Janoša v komentarju na tale issue, pa bomo dodali projekt na repozitorij za zagovore.

[JanKolenc]

Hvala za nasvete. Vse sem upošteval in popravil napake. Mislim, da je sedaj projekt pripravljen za oddajo. Označiti pa žal ne morem, ker jaanos ni med participants.

[jaanos]

Sem dodal povezavo na repozitorij za zagovore, tako da se lahko z @alenFMF domeniš za termin zagovora, če se seveda še nisi. Če boš do takrat še kaj spreminjal, mi javi, da posodobim povezavo.

@JanKolenc, omeniš lahko kogarkoli na GitHubu - tudi, če ni med ponujenimi možnostmi, ko napišeš @, lahko sam napišeš uporabniško ime. Sicer pa sem na tvojem repozitoriju (in vseh ostalih pri predmetu) označil Watch, tako da dobim vsa obvestila.