search

mlr-org / mlr3gallery

Case studies using mlr3
https://mlr3gallery.mlr-org.com
21 stars 9 forks source link

Titanic Separation #41

Closed danielsaggau closed 4 years ago

danielsaggau commented 4 years ago

Split titanic into two parts First part: Data cleansing, Feature engineering, Basic Model Second part: Imputation, Benchmarking (usage of mlr3viz) Renaming into mlr3pipelines titanic - basics mlr3pipelines titanic- Advanced

pat-s commented 4 years ago

@danielsaggau Something went a bit wrong regarding all the files you touched. Please remove the unneeded auxiliary files and always review what your PR does while working at it.

Not sure how you modified all these files. If something is still unclear regarding the general workflow, please tell us so.

berndbischl commented 4 years ago

are we now deleting everything concerning imputation from this post? this doesn't really seem reasonable? that's kinda the main challenge in titanic

berndbischl commented 4 years ago

ah nein, es sind jetzt 2 posts. das ist aber scheiße so. das kostet einfsach zu viel zeit sich das anzugucken. der grund ist einfach:

a) es werden 50 files gepusht. das macht es null übersichtlich b) WARUM enthält der PR nicht 2-3 grobe beschreibung? das wäre normal und sinnvoll.

danielsaggau commented 4 years ago

@berndbischl With respect to the extra files that actually should not have been touched or be there, this is due a mistake a made with distill. In the README it still said create blog and not create post so I thats where all the unnecessary changes come from. I am currently cleaning it up again.

berndbischl commented 4 years ago

@danielsaggau danke für die schnelle antwort. kannst du vielleicht noch eine sache sagen. ist das aus deiner sicht jetzt fertig? weil dann ziehe ich das heute rein

danielsaggau commented 4 years ago

@berndbischl Heute kommen noch 1-2 Sätze zur Beschreibung da der Split bzw die Bestandteile der Einzelnen Post sich geändert haben aber sonst sollte sich nicht mehr viel Inhaltlich zu den eigentlichen Posts ändern.

berndbischl commented 4 years ago

danke, dann mach schnell und ping mich dann nehm ichs rein

danielsaggau commented 4 years ago

@berndbischl, @pat-s der post is ready for review. Jedoch war ich mir immer noch nicht sicher welche Ordner nun hinzugefügt werden müssen bzw wie das ganze mit destill funktioniert. ich habe hier nur die 2 rmd files und die alten Ordner Umbenannt.