'group_by' applied to an object of class "c('plotly', 'htmlwidget') - anyone?

[smoe]

Hello again,
I get this error message for the preparation of section 5.3 of the salmon and the star reports:

<simpleError in UseMethod("group_by"): no applicable method for 'group_by' applied to an object of class "c('plotly', 'htmlwidget')">

Does this ring any bell on your end? The corresponding code is in

pigx_rnaseq/scripts/deseqReport.Rmd.in

Line 421 in 1375b63

group_by(geneId) %>%

in the section generating the boxplots. The volcano plot in 5.1 and the others in 4 are fine.

I am running a self-compiled version on Debian. There are likely differences with the JavaScript and R libraries that I am using. The files generated during the autotesting did not create any R error, though.

My next step would be to prepare a guix installation and compare with the same data set. But you may have an idea already. I found that the quality of the sequencing of this example (picked E-GEOD-49555) is bad (<30 mapped for all except two samples) which may cause a confusion when grouping when one group does not have the gene?!? Just a speculation.

Thanks
Steffen

[borauyar]

@smoe my best guess is a namespace clash issue. group_by function from some other library might be interfering. We will look into this. @dohmjan We should make an explicit call here including the library name where group_by function comes from.

[dohmjan]

Hey! I added the explicit calls for plotly functions. There might have been mix-ups with other libraries as I couldn't reproduce the error using guix. Hope this resolves the problem!
b134627

[smoe]

Thank you tons! I also thought that this is not perfectly reproducible. Please expect feedback next week.

[borauyar]

@smoe do you have any issues with this still?

[smoe]

Sorry, let me revisit your new release (congrats, btw).

[borauyar]

@smoe thank you! Just a warning: the default mapper in the new release is set to "hisat2", which consumes much less memory. This can be configured in the settings.yaml file if you'd like to keep using STAR.

[smoe]

Hisat2 just ran fine, and so does STAR, but in either configuration right afterwards "samtools index" is somehow having an incompatibility with snakemake, reported as #96.
I was a bit surprised by your comment on STAR's memory footprint - had considered this an issue from the past.

Please leave this bug open for another while, since IIRC the problem was not triggered by the regular build tests.

[borauyar]

@smoe shall I close this issue? Is it fixed for you?

[smoe]

My memory is somewhat weak on this - I have escapted to a machine with more memory for my typical pigx-rnaseq runs and there everything completes successfully