r-lib / revdepcheck

R package reverse dependency checking
https://revdepcheck.r-lib.org
Other
101 stars 32 forks source link

NA's in results for `cloud_check()` #295

Closed wch closed 1 year ago

wch commented 3 years ago

I just ran cloud_check() for r-lib/memoise@fb66ecd, and had some NA names and versions in the results. I ran it twice and got these NAs both times. They show up immediately, shortly after the job is started.

> job <- "0e574f15-d4c1-4c8b-ad48-3fba99975576"
> cloud_summary(job)
ℹ Syncing results to revdep/cloud.noindex/0e574f15-d4c1-4c8b-ad48-3fba99975576
ℹ Extracting results
ℹ Comparing results
✓ autota 0.1.3                           ── E: 0     | W: 0     | N: 1                                        
✓ available 1.0.4                        ── E: 0     | W: 0     | N: 1    
✓ BARIS 1.1.1                            ── E: 0     | W: 0     | N: 1    
✓ baytrends 2.0.2                        ── E: 0     | W: 0     | N: 1    
✓ biolink 0.1.7                          ── E: 0     | W: 0     | N: 0    
✓ BWStest 0.2.2                          ── E: 0     | W: 0     | N: 0    
✓ civis 3.0.0                            ── E: 0     | W: 0     | N: 0    
✓ codemetar 0.1.9                        ── E: 0     | W: 0     | N: 1    
✓ conflicted 1.0.4                       ── E: 0     | W: 0     | N: 1    
E NA NA                                  ── E: 1/1   | W: 1/1   | N: 1/1  
✓ covr 3.5.1                             ── E: 0     | W: 0     | N: 0    
✓ cultevo 1.0.2                          ── E: 0     | W: 0     | N: 1    
✓ cymruservices 0.5.0                    ── E: 0     | W: 0     | N: 0    
✓ datarobot 2.17.1                       ── E: 0     | W: 0     | N: 0    
✓ DeLorean 1.5.0                         ── E: 0     | W: 0     | N: 3    
✓ devtools 2.3.2                         ── E: 1     | W: 0     | N: 0    
✓ dm 0.1.10                              ── E: 0     | W: 0     | N: 0    
✓ ecmwfr 1.3.0                           ── E: 0     | W: 0     | N: 0    
✓ eia 0.3.6                              ── E: 0     | W: 0     | N: 0    
✓ ffscrapr 1.2.1                         ── E: 0     | W: 0     | N: 0    
✓ fgeo.x 1.1.4                           ── E: 0     | W: 0     | N: 0    
✓ fishtree 0.3.3                         ── E: 0     | W: 0     | N: 0    
✓ functools 0.2.0                        ── E: 0     | W: 0     | N: 0    
x GetBCBData 0.5                         ── E: 0  +1 | W: 0     | N: 1    
✓ GetQuandlData 0.1.0                    ── E: 0     | W: 0     | N: 1    
✓ googleAnalyticsR 0.8.0                 ── E: 0     | W: 0     | N: 0    
✓ googleAuthR 1.3.1                      ── E: 0     | W: 0     | N: 0    
✓ GSED 2.5                               ── E: 0     | W: 0     | N: 0    
✓ gWidgets2RGtk2 1.0-7                   ── E: 0     | W: 0     | N: 0    
I gWidgets2tcltk 1.0-6                   ── E: 0  +1 | W: 0     | N: 0    
✓ heemod 0.14.0                          ── E: 0     | W: 0     | N: 0    
✓ huxtable 5.1.1                         ── E: 1     | W: 0     | N: 1    
✓ jpmesh 2.0.0                           ── E: 0     | W: 0     | N: 1    
E NA NA                                  ── E: 1/1   | W: 1/1   | N: 1/1  
✓ kimisc 0.4                             ── E: 0     | W: 0     | N: 1    
✓ LAGOSNE 2.0.2                          ── E: 0     | W: 0     | N: 0    
✓ latte 0.2.1                            ── E: 0     | W: 0     | N: 0    
✓ LCAvarsel 1.1                          ── E: 0     | W: 0     | N: 1    
✓ LDATS 0.2.7                            ── E: 0     | W: 0     | N: 0    
✓ m2r 1.0.2                              ── E: 0     | W: 0     | N: 0    
✓ metR 0.9.0                             ── E: 0     | W: 0     | N: 1    
✓ mindicador 0.1.5                       ── E: 0     | W: 0     | N: 1    
✓ mixggm 1.0                             ── E: 0     | W: 0     | N: 1    
✓ mlr 2.18.0                             ── E: 0     | W: 0     | N: 2    
✓ MODISTools 1.1.1                       ── E: 0     | W: 0     | N: 0    
✓ move 4.0.6                             ── E: 0     | W: 0     | N: 0    
✓ nat.templatebrains 1.0                 ── E: 0     | W: 0     | N: 0    
✓ nhdR 0.5.3                             ── E: 0     | W: 0     | N: 0    
E NA NA                                  ── E: 1/1   | W: 1/1   | N: 1/1  
✓ officedown 0.2.0                       ── E: 0     | W: 0     | N: 0    
✓ opencage 0.1.4                         ── E: 0     | W: 0     | N: 0    
✓ OpenML 1.10                            ── E: 0     | W: 0     | N: 0    
✓ osfr 0.2.8                             ── E: 0     | W: 0     | N: 0    
✓ packageRank 0.3.5                      ── E: 0     | W: 0     | N: 0    
✓ phateR 1.0.4                           ── E: 0     | W: 0     | N: 1    
✓ phenocamr 1.1.4                        ── E: 0     | W: 0     | N: 1    
✓ piggyback 0.0.11                       ── E: 0     | W: 0     | N: 0    
✓ pkgdown 1.6.1                          ── E: 0     | W: 0     | N: 1    
✓ pkgsearch 3.0.3                        ── E: 0     | W: 0     | N: 0    
✓ polite 0.1.1                           ── E: 0     | W: 0     | N: 0    
✓ povcalnetR 0.1.1                       ── E: 0     | W: 0     | N: 1    
✓ prcbench 0.9.1                         ── E: 0     | W: 0     | N: 1    
✓ RcppEigenAD 1.0.0                      ── E: 0     | W: 0     | N: 2    
✓ rfishbase 3.0.4                        ── E: 0     | W: 0     | N: 1    
✓ rgho 2.0.1                             ── E: 0     | W: 0     | N: 0    
E NA NA                                  ── E: 1/1   | W: 1/1   | N: 1/1  
✓ rsoi 0.5.4                             ── E: 0     | W: 0     | N: 0    
✓ RSQLite 2.2.2                          ── E: 0     | W: 0     | N: 1    
✓ rtrek 0.3.2                            ── E: 0     | W: 0     | N: 2    
E NA NA                                  ── E: 1/1   | W: 1/1   | N: 1/1  
✓ saeRobust 0.2.0                        ── E: 0     | W: 0     | N: 0    
✓ SamplingStrata 1.5-1                   ── E: 0     | W: 0     | N: 0    
✓ scopr 0.3.3                            ── E: 0     | W: 0     | N: 0    
✓ SIDES 1.15                             ── E: 0     | W: 0     | N: 0    
✓ simfinapi 0.1.0                        ── E: 0     | W: 0     | N: 0    
✓ simfinR 0.2.2                          ── E: 0     | W: 0     | N: 0    
✓ slouch 2.1.4                           ── E: 0     | W: 0     | N: 0    
✓ snotelr 1.0.4                          ── E: 0     | W: 0     | N: 0    
✓ surveillance 1.18.0                    ── E: 0     | W: 0     | N: 2    
✓ SVN 1.0.1                              ── E: 0     | W: 0     | N: 0    
✓ taxadb 0.1.0                           ── E: 0     | W: 0     | N: 1    
✓ TotalCopheneticIndex 1.0.1             ── E: 0     | W: 0     | N: 2    
✓ tradestatistics 1.1                    ── E: 0     | W: 0     | N: 0    
✓ TreeDist 1.2.1                         ── E: 0     | W: 0     | N: 0    
✓ weathercan 0.4.0                       ── E: 0     | W: 0     | N: 2    

Here's what one of the result objects from cloud_results() looks like:


[[66]]
$package
Package 
     NA 

$status
[1] "E"

$old
── R CMD check results ────────────────────────────────────────────────────────────────────────────── NA NA ────
Duration: 

> NA

> NA

> NA

1 error x | 1 warning x | 1 note x

$new
── R CMD check results ────────────────────────────────────────────────────────────────────────────── NA NA ────
Duration: 

> NA

> NA

> NA

1 error x | 1 warning x | 1 note x

$version
[1] NA

attr(,"class")
[1] "rcmdcheck_error"
jimhester commented 3 years ago

They are due to Bioconductor, which we don't currently support

cderv commented 2 years ago

Is there plan to support Bioconductor so that CRAN packages that have a dependency hosted on Bioconductor can be checked ?
We have at least 116 rev deps like that for knitr. (Compared to more then 6000 revdeps, that is not a lot)

Having a way to maybe simplify the report by not included NA could be a first step to avoid this unuseful content though.

FWIW I did that to detect if a package have a bioc dependency

has_bioc_dep <- function(pkg) {
  deps <- pak::pkg_deps(pkg, dependencies = "all")
  any(grepl("bioconductor", deps$sources))
}

but it takes some times to compute on all the ~6600 rev dep to filter out those who won't work on cloud check.