KentonWhite / ProjectTemplate

A template utility for R projects that provides a skeletal project.
http://projecttemplate.net
GNU General Public License v3.0
623 stars 159 forks source link

neither want to use data.table nor tibble. Can I turn both off? #235

Closed alsmnn closed 6 years ago

alsmnn commented 6 years ago

Report an Issue / Request a Feature

I'm submitting a (Check one with "x") :


Issue Severity Classification -

(Check one with "x") :

Expected Behavior

I want to choose which data frame should be transformed to a tibble

Current Behavior

Every dataframe is transformed to a tibble

Steps to Reproduce Behavior

Load the project

library(ProjectTemplate); load.project()

Screenshots
Version Information
Possible Solution

make it possible to turn data.table- and tibble-conversion off

alsmnn commented 6 years ago

The reason why I´m asking for this is that several bioconductor packages need the row.names(x) value in order to create the bioconductor classes like SummarizedExperiment or ExpressionSet from Biobase.

Cheers

Hugovdberg commented 6 years ago

Currently it is not possible to disable the conversion to tibble. It was actually a conscious decision to implement it this way. The whole idea of ProjectTemplate is to build a framework that provides sensible defaults, with limited configuration possibilities (convention over configuration). I'm under the impression that the majority of people will not run into this issue you experience, and to paraphrase Hadley Wickham, if the row names actually tell something about the data it should be a normal column.

I would suggest that for this very specific use case you include a munge script that converts the data back to a normal dataframe.

alsmnn commented 6 years ago

I agree with you and Hadley Wickham on the rowname issue, unfortunately the bioconductor packages didn´t have this in mind. Maybe in the future. Than I will add this mungescript as you suggest it.

Best regards edit: typo

Hugovdberg commented 6 years ago

That's rather unfortunate indeed, let's hope the bioconductor people will fix this for you ;-) closing this issue for now.