@eisenman the change in develop branch has not been uploaded to GitHub, is this not automatically synchronized?. So the user who reported the bug still has the same problem. It would be good to merge into master as well

now test case in test-report.R

very simple fix in rankingHeatmap.challenge
@eisenman can this be merged into master branch?

graph had only been used for networks, I guess version 1.62 is sufficient

additional test in test-blobPlotStabilityByAlgorithm for the case "one task out of 3 tasks contains >1 test cases". in this case bootstrap() gives result for this remaining task and stability() only produces plot with this task. @eisenman could you please check and close?

@eisenman yes, should be sufficient
only one case per task covered in bootstrap() now see T28202

bootstrap() now gives error if all tasks only contain 1 test case and a message if some tasks contain only 1 test case (tasks with 1 test case are excluded from bootstrapping)
test-bootstrap.R contains tests. @eisenman could you please check test

task names ( names(x$matlist) ) had been used for facet titles instead of algorithm names ( stored in "ordering").

platform is output with Sys.info(); linux and Mac systems are equal and windows may differ

good idea!
otherwise you could use a platform dependent expect_equal but I think your idea is sufficient

would mention it there at least though because bootstrapping may be time consuming

don't think its necessary. there is anyway ordering applied to it in later stages.
otherwise ranking step (rank.aggregated.list()) could always do the ordering directly (then the test would check whether ordering is working). As mentioned before I avoided such change before the release because of (very unlikely) risk of breaking something in the first level hierarchy

I do see those as well on Mac

R 4.0.2.
testthat previously 2.3.2, now 3.0.1.
the issue is however not connected to test_that I see that in the first 2 tests in aggregate-then-rank the ordering in the data set is not used but sorted

seems to not fail on windows machines. but does on my Mac (and thus might also on linux). minor issue, defer to post 1.0.0

would defer to after release

please close if ok

added to documentation of as.challenge():
(arg annotator:) If multiple annotators annotated the test cases, a string specifying the name of the column that contains the annotator identifiers. Only applies to rang-then-aggregate. Use with caution: Currently not tested.

added

podium(ranking)

to vignette.
podium() does have a different syntax for layouting (it is not ggplot2 but base graphics) but vignette does not describe layouting for ggplot2 plots either

report() does not know what type of consensus ranking method had been used.
I added sentence
"Consensus ranking according to mean ranks across tasks if method="euclidean" where in case of ties (equal ranks for multiple algorithms) the average rank is used, i.e. ties.method="average"."
to help for consensus() and would suggest to also add this to the vignette/readme, but not to report. Could someone of you please take care of this?

@eisenman do you want to do this with roxygen?

kept select.if(), winner(), extract.workfolow and compareRanks()
and removed everything not supported anymore.
as.warehouse (benchmarkUtils) is not exported, recommend to leave because this may come handy for specific situations

removed second()

What is the problem?
in the extend of the vignette it would be simply

legend now always at bottom and dynamically adjusts to number of algorithms and algorithm name length.
Took me almost the day...