Open beckerbenj opened 1 year ago
Suggestions
Simple Comparisons gDiff cDiff trend (Spalte)
Double comparison gDiff_cDiff (currently: crossDiff_of_groupDiff) cDiff_gDiff (does currently not exist) gDiff_trend (Trend column in gDiff row) trend_gDiff (currently: trendDiff_group) cDiff_trend (Trend column in cDiff row) trend_cDiff (currently trendDiff_cross)
Triple comparison gDiff_cDiff_trend (Trend column in row crossDiff_of_groupDiff) cDiff_gDiff_trend (does currently not exist) gDiff_trend_cDiff (currently trendDiff_cross) cDiff_trend_gDiff (does currently not exist) trend_cDiff_gDiff (does currently not exist) trend_gDiff_cDiff (does currently not exist)
Challenges
Suggestions @nickhaf:
Tidy data principle: each information has a designated cell
Open question:
Backwards compatability?
report2()
? But maintainability?
Clean cut?
Meeting with @beckerbenj , @nickhaf, @buchjani
Structure more or less discussed and agreed upon; implementation lead @weirichs
@nickhaf: Baut Beispiel aus
@nickhaf stellt Termin ein mit SW, JB
Termin hat stattgefunden, @weirichs ist an der Umsetzung
Beispiel für die neue Struktur: P:\Methoden\02_R_Pakete\eatRep\Outputstruktur
@weirichs hat Outputstruktr vorgestellt, Austausch dazu im Gange
Termin zur Userbefragung wurde angesetzt
Termin hat stattgefunden. Es gab keine klare Präferenz. SW setzt probeweise Text und "Formel"-Schreibweise für Labels um.
NH erstellt kurzen Absprache-Termin dazu, um Ergebnisse und nächste Schritte zu sammeln.
SW hat gemäß Umfrageergebnis was umgesetzt, wir schauen uns das jetzt mal an.
@nickhaf hat schon drüber geschaut @sachseka schaut noch -- @weirichs übersetzt auf ENG
Auch nicht variierende Kodierungsvariable wird in die label-Spalte mit aufgenommen. Labels auch für nicht comparisons.
Sebastian ist noch am basteln
@weirichs ist noch an der techn. Umsetzung
Funktionalität steht, @weirichs räumt Code auf
Änderung ist soweit fertig, alte Version soll noch mit Versionsnummer konserviert werden, weil danach ein Breaking change kommt. @buchjani reviewed.
alte reportingfkt ist jetzt deprecated, geht noch, aber hat nicht mehr denselben Output. @buchjani Review steht noch aus. Kommt das noch? oder soll @weirichs ohne Review pushen? eher nicht. @buchjani machts heute!
@buchjani hat gereviewed , @weirichs überarbeitet
Currently the output structure of
report()
is rather messy and difficult to comprehend. See below file for an overview and suggestions for an overhaul.This should be restructured and properly documented. Also be aware of further processing by eatPlot