ropensci/visdat

make `vis_guess` fast

Offen

#14 geöffnet am 06.03.2016

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)R (47 Forks)github user discovery
Priority 2enhancementhelp wantedmaintenance

Repository-Metriken

Stars
 (462 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Currently vis_guess is a bit slow and it'd be nice to make it faster.

We found that using vapply was much faster than using purrr, which is a bit of a shame as I think that the current code doesn't read as nice as purrr code (dare I say it, it doesn't purrr).

The way I see it we have two options to really speed things up.

  1. Write some C++ code based on the readr::::CollectorGuess code, so that it deals with a vector, rather than a single element.
  2. Use one of the functions from the parallel library, like mcapply.

Contributor Guide

Research-Richtung
Profiliere die aktuelle `vis guess` Funktion, um Engpässe zu identifizieren. Erwäge die Implementierung einer C++ Version mit Rcpp für vektorisierte Schätzungen oder erkunde die Parallelisierung mit Funktionen aus dem `parallel` Paket wie `mclapply`. Vergleiche die Verbesserungen mit der aktuellen Implementierung durch Benchmarks.
Tech Stack
rcpp
Domain
data visualizationperformance
Issue Type
Performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
RGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60