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ček projekta #4

Open
laravidmar opened this issue Feb 19, 2020 · 4 comments
Open

Zaključek projekta #4

laravidmar opened this issue Feb 19, 2020 · 4 comments

Comments

@laravidmar
Copy link
Owner

Pozdravljeni,
projekt sem zaključila, je potrebno še kaj popraviti?
Lep pozdrav,
Lara Vidmar

@timotejvesel
Copy link

Pri izrisu zemljevidov izpiše neko sporočilo. Temu se lahko izogneš tako, da tam kjer ju kličeš dodaš message=FALSE:
image

jaanos added a commit to jaanos/APPR-zagovori that referenced this issue Feb 21, 2020
@jaanos
Copy link

jaanos commented Feb 21, 2020

Dodal sem 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 spreminjala, mi javi, da posodobim povezavo.

Kot je omenil @timotejvesel, s parametrom message=FALSE preprečiš izpisovanje sporočil v poročilo, vendar imaš na dveh mestih namesto message napisano massage, tako da se pri meni eno sporočilo še vedno izpiše. Poleg tega svetujem, da pri izpisu grafa graf_evropa dodaš še parameter echo=FALSE, da se v poročilo ne izpiše ukaz za izpis grafa.

Poleg tega imam še sledeče pripombe estetske narave:

  • Pri grafih transporta blaga in potnikov se na osi y pojavljajo oznake v eksponentni notaciji. Da se jih znebiš, lahko podatke ustrezno deliš (npr. s 1000 - potem ustrezno popravi še oznake osi y), ali pa narediš podobno kot pri Grafi- vizualizacija evawinkler/APPR-2019-20#3 (comment). Poleg tega svetujem še, da ročno nastaviš še oznake osi x, da se znebiš neželenih navednic, ter poravnaš imena držav (pri element_text dodaj parameter hjust=1).

  • Pri napovedih rasti poskrbi, da bodo na osi x celoštevilske letnice - grafoma lahko prišteješ npr. scale_x_continuous(breaks=seq(2010, 2020, 2)), da se izpiše vsaka druga letnica.

@laravidmar
Copy link
Owner Author

Hvala za odgovor, sem popravila. Prosim, če mi posodobite povezavo.

jaanos added a commit to jaanos/APPR-zagovori that referenced this issue Feb 24, 2020
@jaanos
Copy link

jaanos commented Feb 24, 2020

Sem posodobil.

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

3 participants