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
timotejvesel opened this issue Jun 30, 2019 · 5 comments
Open

Zaključek projekta #3

timotejvesel opened this issue Jun 30, 2019 · 5 comments

Comments

@timotejvesel
Copy link
Owner

Projekt sma dokončala.

Pri dodajanju komentarjev v aplikaciji v zavihku Civilen diskurz o vojnah je čas komentarja dve uri nazaj kot pa je dejanski čas ob objavi komentarja (npr. 18:00 namesto 20:00). Mogoče veste v čem je problem?

@jaanos
Copy link

jaanos commented Jul 1, 2019

Projekt izgleda v redu, tako da bom dodal povezavo na repozitorij predmeta, z @alenFMF pa se lahko domenita za termin zagovora, če se seveda še nista. Če bosta do takrat še kaj spreminjala, mi javita, da posodobim povezavo.

Kar se prikaza časa tiče, je stvar taka, da datatable privzeto prikaže čas v časovni coni UTC (zato Z na koncu). Lahko poskrbita za prikaz glede na lokalne nastavitve:

output$komentiranje <- DT::renderDataTable(DT::datatable(najdi.komentar()) %>%
                                             DT::formatDate("Cas", method="toLocaleString"))

Lahko pa enostavno stolpec Cas pretvorita v nize ustrezne oblike (npr. s strftime), vendar bo to morda lahko vplivalo na to, ali se bodo ti časi pravilno urejali.

Imam še nekaj pripomb:

  • Tabeli povzroci in sodelovanje_koal nimata nastavljenega glavnega ključa. V obeh primerih bi bilo smiselno, da ta sestoji iz obeh tujih ključev.
  • V tabeli komentar je stolpec ime odveč, saj ga lahko dobita že iz tabele uporabnik.
  • Pri pregledu vojn vedno izpisujeta vojne, kjer število žrtev ni znano. Svetujem, da te obravnavata, kot da je število žrtev enako 0 (pomagata si lahko s COALESCE).
  • Pri statistiki po sodelujočih se spodaj izpiše napaka, da spremenljivka vojne_po_letih ni definirana.

@timotejvesel
Copy link
Owner Author

Sem popravil. Prosim da posodobite.

Za časovno cono sem naredil

 output$komentiranje <- DT::renderDataTable(DT::datatable(najdi.komentar())%>%
                                              DT::formatDate("Cas", method="toLocaleString"))

kot ste predlagali. Vendar pri vojnah, ki še nimajo nobenega komentarja javi napako Warning: Error in name2int: You specified the columns: Cas, but the column names of the data are Stack trace (innermost first):

@jaanos
Copy link

jaanos commented Jul 1, 2019

Da ne pride do take napake, se lahko v najdi.komentar (pred vračanjem rezultata) doda npr.

validate(need(nrow(komentarji) > 0, "Ni komentarjev."))

Tako se bo izpisalo podano sporočilo v primeru, ko je razpredelnica komentarji prazna.

Bom počakal, da še to uredita, pa bom posodobil.

@timotejvesel
Copy link
Owner Author

Sem popravil.

jaanos added a commit to jaanos/OPB that referenced this issue Jul 1, 2019
@jaanos
Copy link

jaanos commented Jul 1, 2019

V redu, 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

2 participants