> Constantin Ahlmann-Eltze via R-devel
> on Mon, 10 Aug 2020 10:05:36 +0200 writes:
> Thanks Ben for verifying the issue. It is always reassuring to hear
> when others can reproduce the problem.
> I wrote a small patch that fixes the issue
>
Kasper,
The tradition so far has been to package all UCSC human genomes since
hg17. We could also start producing BSgenome packages for other non-UCSC
Human assemblies. We just need to draw a line somewhere. If there is a
need for it we can make BSgenome.Hsapiens.NCBI.GRCh37.p13 available, as
Well, the presence of two mitochondrial genomes is to fix a mistake by
UCSC. I can appreciate the importance of representing this mistake when you
build off UCSC. But it strikes me as not actually representing the h37
version of the genome, and it seems to me that we want such a
representation in
> Gabriel Becker writes:
> I added that so I can look at the proposed fix and put it or something
> similar in bugzilla for review final review.
> Apologies for the oversight.
Fixed now with
-while(as.character(bdexpr[[1L]]) == "{")
+while(is.call(bdexpr) &&
> Duncan Murdoch writes:
> On 19/08/2020 12:26 p.m., Toby Hocking wrote:
>> Hi the reference to R Internals
>> https://cran.r-project.org/doc/manuals/r-release/R-ints.html#Tools
>> in ?check (PkgUtils.Rd in utils package) is stale. Here is my proposed
>> patch (use named reference rather than
