Bioconductor / ExperimentHub

Client to access ExperimentHub resources
https://bioconductor.org/packages/ExperimentHub
9 stars 13 forks source link

devel branch error ... GenomicAlignments issue? #30

Closed vjcitn closed 1 year ago

vjcitn commented 1 year ago
----
ERROR EVENTS:
----
checking re-building of vignette outputs ... ERROR
Error(s) in re-building vignettes:
  ...
--- re-building ‘ExperimentHub.Rmd’ using rmarkdown

Quitting from lines 141-142 [dm3] (ExperimentHub.Rmd)
Error: processing vignette 'ExperimentHub.Rmd' failed with diagnostics:
failed to load resource
  name: EH166
  title: ERR188297
  reason: require("GenomicAlignments") failed: use BiocManager::install() to install package?
--- failed re-building ‘ExperimentHub.Rmd’

SUMMARY: processing the following file failed:
  ‘ExperimentHub.Rmd’

Error: Vignette re-building failed.
Execution halted
lshep commented 1 year ago

Where are you seeing this or get this from? I don't see this on a build report anywhere?

mtmorgan commented 1 year ago

hello hello I have seen this in recent R-patched. It occurs when a package is used in the vignettes, but not declared in the suggests field of the description file.

lshep commented 1 year ago

ah so related to the new variable declaration on the builders. I think this is legit if you are using package data you should have the package loaded and declared appropriately.

lshep commented 1 year ago

what is interesting is we don't see this in release_3_17 even tho the variable is active there too

lshep commented 1 year ago

https://github.com/Bioconductor/ExperimentHub/commit/b24eca5e2bee701f42542b9eca51d505cd555a05