[Bioc-devel] problems understanding build error

[Lluís Revilla]

Dear Michael,

The error means that length(which(cond)) > 1
I think this is a bug on BiocCheck on the internal function
.getTokenTextCode (findSymbols.R L12
<https://github.com/Bioconductor/BiocCheck/blob/222e4693a947673ef16d9e6982e7f1c14e2c705f/R/findSymbols.R#L12>
).
It checks if any condition is met not if it is met only once.

I hope this helps,

Best,



On Fri, 26 Jul 2024 at 11:29, Michael Shapiro <michael.shapiro using crick.ac.uk>
wrote:

> We are currently working through the submission of a new package,
> CatsCradle (issue #3452).  We get a build error from the Coding Practices
> section of the BiocCheck tests, namely
>
>
> Error in seq.default(which(cond) - length(notLookback), which(cond)) :
>   'from' must be of length 1
>
> For whatever reason, we do not get this error when running BiocCheck
> locally.
>
> We have consulted
>
> https://bioconductor.org/packages/devel/bioc/vignettes/BiocCheck/inst/doc/BiocCheck.html#coding-practices-checks
> and the closest we can find is the recommendation to use seq_len(N) in
> place of things like 1:N.    We are now consistently using seq_len for
> sequences that begin with 1 and seq(from=a,to=b) for sequences beginning at
> other values but this has not resolved the error.
>
> Any assistance with this would be very much appreciated.
>
> The Francis Crick Institute Limited is a registered charity in England and
> Wales no. 1140062 and a company registered in England and Wales no.
> 06885462, with its registered office at 1 Midland Road London NW1 1AT
>
>         [[alternative HTML version deleted]]
>
> _______________________________________________
> Bioc-devel using r-project.org mailing list
> https://stat.ethz.ch/mailman/listinfo/bioc-devel
>

	[[alternative HTML version deleted]]