setDT could be much simpler

Aperta
#4,476 2 commenti 1 reazione 1 assegnatario Vedi su GitHub

@jangorecki ci sta già lavorando.

Dal 22/5/2020.

  • #4477 di @jangorecki — aperta

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

benchmark

setDT function already grown pretty fat. AFAIK it was meant to be a low overhead function. Making it low overhead now would be challanging, or even impossible, without removing some of its features. Therefore it seems reasonable to introduce new argument, lets call it asis, that will make as few as possible, which is to set:

  • class
  • rownames
  • overalloc

Such a change would be helpful in case if someone wants to benchmark DT operations but for some reason would like to include time of as.data.table / as.data.frame (time of an extra deep copy). setDT already removes extra deep copy, but they have other stuff, that is unnecessary in a case of just changing class.

Lingua principale
R
Stelle
3.9k
Fork
1.1k
Merge medio
14h 4m
PR unite (30g)
4

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Rdatatable/data.table

Tutte le issue di Rdatatable/data.table

Issue simili

Altre issue su R

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.