repr icon indicating copy to clipboard operation
repr copied to clipboard

Use pander and xtable for matrix, data.frame and so on

Open jankatins opened this issue 9 years ago • 3 comments

@daroczig raised the very valid point in https://github.com/Rapporter/pander/pull/251, that we should not (re)implement how we do html and MD conversation of standard objects in repr:

Sorry for the silly question, but before figuring out the optimal next steps in short & long terms, why are you writing and maintaining the code for transforming R objects into HTML, tex or markdown? There's xtable and a bunch of other packages out there doing this and eg pander for markdown. So instead of having https://github.com/IRkernel/repr/blob/master/R/repr_matrix_df.r, you could simply call xtable for HTML or tex, and eg pander in a markdown notebook. Eg the pander way of writing reproducible reports is that you never call pander in your Rmd, as pander will be automatically applied on all the R objects.

jankatins avatar Mar 15 '16 08:03 jankatins

mainly because at the time, i just wrote a bunch of reprs for several kinds of objects and didn’t know that there were packages already doing that specifically for tabular data.

we could use them but then we’d have to use an option adapter (i.e. translate repr options to the specific packages’ equivalents)

flying-sheep avatar Mar 15 '16 08:03 flying-sheep

IMO we could either

repr_markdown.data.frame <- function(x)
   paste(capture_output(pander.data.frame(x)), collapse="\n")

or do a

repr_markdown.data.frame <- pander.data.frame

when/if pander supports a return mode...

Same for xtable...

jankatins avatar Mar 15 '16 10:03 jankatins

Re options: why not simple tell them the pander options? Would make things like https://github.com/IRkernel/repr/issues/14 much easier...

jankatins avatar Mar 15 '16 10:03 jankatins