Unstated namespace dependency on data.table::head

[andrewrech]

sleuth/R/sleuth.R

Line 800 in 2cbb287

#' head(sleuth_genes) # show info for first 5 genes

Error: package or namespace load failed for ‘immune.ml’:
object 'head' not found whilst loading namespace 'sleuth'
Execution halted

• checking whether the namespace can be loaded with stated dependencies ... WARNING
  Error: object 'head' not found whilst loading namespace 'sleuth'
  Execution halted

A namespace must be able to be loaded with just the base namespace
loaded: otherwise if the namespace gets loaded by a saved object, the
session will be unable to start.

Thanks

[warrenmcg]

Hi @andrewrech,

Sorry for delay in responding. Are you still experiencing this issue? Could you describe more about what you are doing, as I have never seen this before in any of the times that I used sleuth within R?

[andrewrech]

Yes I am. Thank you. You need you specify the namespace dependency for head. I am depending on sleuth in an R package:
data.table::head

[andrewrech]

* checking whether the namespace can be loaded with stated dependencies ... WARNING
Error: object 'head' not found whilst loading namespace 'sleuth'
Execution halted

A namespace must be able to be loaded with just the base namespace
loaded: otherwise if the namespace gets loaded by a saved object, the
session will be unable to start.

Probably some imports need to be declared in the NAMESPACE file.

[warrenmcg]

Hi @andrewrech, I discovered that there was an API error. The method is supposed to return a data.frame but was instead a dplyr object. Once this API error is corrected, there will not be any need to update the namespace, as head for data frames is built-in. This mistake will be fixed in the upcoming release. I'm submitting a pull request with miscellaneous changes today.

[andrewrech]

Thanks! @warrenmcg

[pimentel]

ugh. this is a really annoying bug. thank you both, @andrewrech @warrenmcg!

this fix is now in the development version and will be in the next release.

[andrewrech]

@pimentel sorry to bring this up again...still broken on HEAD :-)

─  checking whether the package can be loaded with stated dependencies ...Error: package or namespace load failed for ‘<Andrew's package>’: (3.3s)
    object 'head' not found whilst loading namespace 'sleuth'
   Execution halted

   It looks like this package (or one of its dependent packages) has an
   unstated dependence on a standard package.  All dependencies must be
   declared in DESCRIPTION.
   See section ‘The DESCRIPTION file’ in the ‘Writing R Extensions’
   manual.

This is clearly exported in your NAMESPACE...I don't get it

[warrenmcg]

Well, it's probable that the same issue is happening: a data.table or dplyr object is being manipulated using head and so the checks is complaining that the specific version of head isn't declared. I'm not sure how quickly I can get to this, but I'll report back as soon as I can.

[andrewrech]

Hmm, well, is devtools::check actually aware of this? It's not running any code so how could this type checking happen?

I just forked sleuth and exported head <- utils::head so my package builds...perhaps we should collectively stop wasting time in the bowels of R CMD check. My issue is solved, feel free to close.