Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zaključevanje projekta #6

Open
Klarapenko opened this issue Aug 31, 2021 · 6 comments
Open

Zaključevanje projekta #6

Klarapenko opened this issue Aug 31, 2021 · 6 comments

Comments

@Klarapenko
Copy link
Owner

Pozdravljeni,

Mislim da sem s projektom zaključila, če pa imate še kakšno opombo ali komentar pa z veseljem popravim.

Hvala

@jaanos
Copy link

jaanos commented Aug 31, 2021

Projekta ne morem zagnati na Binderju - kolikor vidim, pride do napake tukaj:

samomori.m.z. <- S_SAMOMORI %>%
group_by(spol, leto) %>%
summarise(vsota=sum(vrednost, na.rm=TRUE))
pivot_wider(names_from = spol, values_from = vsota) %>%
mutate(
deležŽensk=round(Females/(Males + Females), 2),
deležMoški=round(Males/(Males + Females), 2)
)

Izgleda, kot da manjka %>% v vrstici s summarise.

@Klarapenko
Copy link
Owner Author

Hvala sem popravila, upam da zdaj deluje?

@jaanos
Copy link

jaanos commented Sep 2, 2021

Projekt se še vedno ne zažene - da bo deloval, bo potrebno dvakrat pobrisati %>% View():

PREB <- PREBIVALSTVO %>% filter(letoo >= 2011 & letoo <= 2017) %>% View()
sapply(PREB, class)
#s tem ukazom dobim tabelco s številom preb za vsa leta samo za Albanijo
PREB %>% filter(DRZAVA == "Albania") %>% View()

Da se ti v zadnjem razdelku ne bodo izpisovala sporočila, v glavo bloka kode, kjer kličeš analiza.r, dodaj še parametre echo=FALSE, message=FALSE, warning=FALSE (tako kot drugod). Treba bo popraviti tudi grafe z napovedmi - napovedne točke namreč računaš tako, da eksplicitno nastaviš prosti člen na 0 (pri formuli, podani pri lm, imaš še + 0), česar pa pri geom_smooth ne storiš. Da bodo napovedne točke ležale na napovedni krivulji, to poenoti (svetujem, da pri lm odstraniš + 0). Napovedi bi sicer bilo smiselno delati za leta, za katera nimaš podatkov (tj., od 2019 naprej).

@Klarapenko
Copy link
Owner Author

Sem popravila in se mi projekt zdaj zažene. Prav tako pa sem uredila nekaj pomanjkljivosti(države zapisala v slovenščini namesto angleščini, tabelam odstanila vrstice, kjer je bil pri državi podatek NA) in dodala shiny.

@jaanos
Copy link

jaanos commented Sep 3, 2021

V redu, bom dodal povezavo do projekta na repozitorij za zagovore. Če se še nisi, se lahko z @alenFMF domeniš za termin zagovora. Če boš do takrat še kaj spreminjala, mi javi, da posodobim povezavo.

jaanos added a commit to jaanos/APPR-zagovori that referenced this issue Sep 3, 2021
@Klarapenko
Copy link
Owner Author

Najlepša hvala za vso pomoč in dogovorjeno, sporočim če bom še kaj spreminjala.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants