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 #3

Open
majaabraham opened this issue Feb 25, 2021 · 5 comments
Open

Zaključek projekta #3

majaabraham opened this issue Feb 25, 2021 · 5 comments

Comments

@majaabraham
Copy link
Owner

majaabraham commented Feb 25, 2021

Menim, da sem z delom na projektu zaključila. Zanima me, ali je potrebno še kaj dodati ali popraviti.

@jaanos
Copy link

jaanos commented Feb 26, 2021

Poročilo deluje, tako da 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.

Edina pripomba, ki jo imam, je ta, da je besedilo na grafih zelo majhno in ga je težko brati, tako da svetujem, da ga nekoliko povečaš.

jaanos added a commit to jaanos/APPR-zagovori that referenced this issue Feb 26, 2021
@majaabraham
Copy link
Owner Author

Zahvaljujem se za komentar in sporočam, da sem besedilo na grafih povečala, tako, da so sedaj lažje berljivi.

@jaanos
Copy link

jaanos commented Feb 26, 2021

V redu, bom posodobil povezavo.

Zdaj vidim, da se ti na dveh grafih (zadnji spletni nakup po starostnih skupinah ter odstotek kupcev, ki so navedli težavo) pojavljajo letnice z decimalko - da se tega znebiš, grafoma prištej npr. scale_x_continuous(breaks=seq(2010, 2019, 3)), da se na osi x izpiše vsako tretje leto od 2010 do 2019. Pri zadnjem grafu svetujem tudi, da oznake na osi x poravnaš sredinsko glede na črtice (k element.text pri axis.text.x dodaj še parameter vjust=0.5).

jaanos added a commit to jaanos/APPR-zagovori that referenced this issue Feb 26, 2021
@majaabraham
Copy link
Owner Author

Sem popravila prikaz letnic.

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

jaanos commented Feb 27, 2021

V redu, sem posodobil povezavo.

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