search

ululalbab / google-motion-charts-with-r

Automatically exported from code.google.com/p/google-motion-charts-with-r
0 stars 0 forks source link

0.4.4 broke knitr functionality (or the other way around) #25

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Creating a gvisTable in knitr

What is the expected output? What do you see instead?
Before the update the table would render in chrome, safari, and firefox. Now 
the html page shows:
<!– jsHeader –>
<!– jsChart –>
<!– divChart –>

What version of the product are you using? On what operating system?
MacOSX 10.8.4
R 3.0
knitr 1.4.1
googleVis 0.4.4
Please provide any additional information below.
example code:

```{r gvis table, results='asis'}
Table <- gvisTable(ddply.df, options = list(width= 1100, height = '100%'))
# Display chart
print(Table, 'chart')

Original issue reported on code.google.com by skycondi...@gmail.com on 28 Aug 2013 at 6:24

GoogleCodeExporter commented 9 years ago 
From knitr forum: https://groups.google.com/forum/#!topic/knitr/BhB2pwL_SqA

Original comment by skycondi...@gmail.com on 28 Aug 2013 at 10:55

GoogleCodeExporter commented 9 years ago 
Indeed, it appears that version 0.4.4 introduced a bug in googleVis with knitr. 
It seems that subtle changes to the indentation caused the issue. I have 
updated the package and uploaded it on out project site:
http://code.google.com/p/google-motion-charts-with-r/downloads/list
I believe it works fine now. Could you please test as well and report back.

Original comment by markus.g...@googlemail.com on 29 Aug 2013 at 6:54

GoogleCodeExporter commented 9 years ago 
I just installed 0.4.4 from source and it looks like it's fixed. Does this mean 
that you have to resubmit 0.4.4 to CRAN now?

Thanks for your help Markus!

Original comment by skycondi...@gmail.com on 29 Aug 2013 at 2:40

GoogleCodeExporter commented 9 years ago 
No, I will have to submit version 0.4.5.

Original comment by markus.g...@googlemail.com on 29 Aug 2013 at 2:55