-
Notifications
You must be signed in to change notification settings - Fork 2
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
Verbessern des Setup Codes in lernplattform.Rmd #17
Comments
Agnes |
Ich hatte es heute morgen schon in eurem Pull-Request kommentiert, weil der jetzt schon gemergt wurde und nicht mehr sichtbar ist, hier noch einmal: Gibt es einen Grund, dass die Daten jedes Mal neu gezogen und gecleant werden? Aus Nutzer:innen-Perspektive ergibt das m.E. keinen Sinn, für die Reproduzierbarkeit ist es auch nicht optimal (Online-Quellen können z.B. wegbrechen). Falls nicht, mein Vorschlag: Man könnte die Daten 1x ziehen, cleanen und ablegen - und die Skripte dafür zu Dokumentationszwecken in einem separaten Ordner ablegen, der aber beim Bauen der Seite nicht jedesmal angesteuert wird. |
Mit „Daten neu ziehen“ meinst du das Neuladen der Daten in den einzelnen Sitzungen? Das ist pädagogisch tatsächlich meiner Meinung nach sinnvoll |
Ich meine nicht das Cleanen, das die Teilnehmenden selbst durchführen. Ich meine den Setup-Code, der in "lernplattform.Rmd" noch vor der ersten Lektion auf serverseitig durchgeführt wird. ### Audit Datensatz laden
audit <- rio::import('https://raw.githubusercontent.com/CorrelAid/lernplattform/main/daten/bffp2019_audit_by_country_and_company.csv')
# EU-Asien Audit Plastik Datensatz erstellen
audit_eu_asia <- audit %>%
dplyr::filter(continent == "Europa" | continent == "Asien") %>%
dplyr::filter(n_pieces != 0) %>%
dplyr::group_by(continent, country) %>%
dplyr::summarise_all(funs(n_distinct(.))) %>%
dplyr::select(continent, country, "n_types" = plastic_type) Soweit ich sehen kann, sehen die Teilnehmenden diesen Code nie. Im Grunde wird also das immer gleiche Datenset auf die immer selbe Art vorbereitet. Könnte man die Daten dann nicht gleich in der benötigten Form ablegen? Oder verstehe ich etwas falsch? |
Ah, ich sehe was du meinst - danke für die Erklärung! In dem Fall wäre ich auch für eine Auslagerung 😊 |
Hi David, gute Anmerkung, das sehe ich genauso. Ich kann mich dem gerne annehmen. Übrigens: Du kannst auch bereits gemergete Pull Requests sehen indem du unter Pull Requests in der Zeile oben is:open entfernst und mit Enter bestätigst. |
z.B. auslagern in Funktionen
The text was updated successfully, but these errors were encountered: