phylo42 / PEWO

Phylogenetic Placement Evaluation Workflows : Benchmark placement software and different reference trees
MIT License
12 stars 9 forks source link

Troubles with eval_resources_plots.R running the pipeline with single software #11

Closed nromashchenko closed 3 years ago

nromashchenko commented 3 years ago

I just ran into an issue with eval_resources_plots.R for a run that only tests epa. Looks like the script presupposes that results for rappas and others must always be there; is there some easy way of fixing/hacking this to work? See warnings and erros below.

Cheers, Pierre

[1] "OP:hmmer-align"
Warning message:
In analyses["epa"] <- c("hmmer-align", "epa-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["epang_h1"] <- c("hmmer-align", "epang-h1-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["epang_h2"] <- c("hmmer-align", "epang-h2-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["epang_h3"] <- c("hmmer-align", "epang-h3-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["epang_h4"] <- c("hmmer-align", "epang-h4-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["pplacer"] <- c("hmmer-align", "pplacer-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["apples"] <- c("hmmer-align", "apples-placement") :
  number of items to replace is not a multiple of replacement length
Warning message:
In analyses["rappas"] <- c("ansrec", "rappas-dbbuild", "rappas-placement") :
  number of items to replace is not a multiple of replacement length
Error in fix.by(by.x, x) : 'by' must specify uniquely valid columns
Calls: merge ... merge.default -> merge -> merge.data.frame -> fix.by
Execution halted 
blinard-BIOINFO commented 3 years ago

Fixed in 577d981ed0fea0b33d7cc47264eac87699f9964a . Waiting for P. Barbera to validate it in his own developements.

blinard-BIOINFO commented 3 years ago

Need to add block for considering Appspam in this script too (last plots mixing software).

blinard-BIOINFO commented 3 years ago

Closed.