Gianluca Baio

Results 90 comments of Gianluca Baio

@n8thangreen **FOR WHEN YOU'RE BACK** (and with no massive pressure on the timeline!). BUT: can we look at the [TODO.md](https://github.com/giabaio/BCEA/blob/dev/TODO.md) file and check whether any of the suggestions there are...

Thanks @n8thangreen. Re the issues, do you need me to do anything? Would you like me to actually create separate issues for each point? If so, I'll do that later!...

OK -- I've updated the Actions and the _pkgdown settings so now all the checks work. I've also checked the dev version of the package in Rstudio and all seems...

Thanks, @teunbrand. We've been working on some relatively major changes, which I have just integrated into the `main` repo. I'll look into this and hopefully be able to fix quickly.

OK, so. I can see the error: ``` plot(he,graph = "gg",ICER_size=5.5,text=list(size=rel(1.25))) Error in `method(update_ggplot, list(ggplot2::theme, ggplot2::ggplot))`: ! Can't merge the `text` theme element. Caused by error in `method(merge_element, list(class_any, class_any))`:...

Thanks. Let us have a go --- I had a *very* quick look earlier and it looked like the theme was kind of set. In the old `ggplot`, the list...

So: the offending code acts to create a multiplot made by 4 different types of graphs, each called by a BCEA function. For instance, the first of the 4 panels...

This **seems** to be doing it... (give or take... I've only modified the ceplane.plot for now and the legend is not quite right -- but I suspect I'm missing some...

@teunbrand we have something that is at least OK to pass the tests and behave with the new version of `ggplot2`. We're hoping to find some time to make that...

[like] Baio, Gianluca reacted to your message: ________________________________ From: Teun van den Brand ***@***.***> Sent: Monday, June 23, 2025 11:16:03 AM To: giabaio/BCEA ***@***.***> Cc: Baio, Gianluca ***@***.***>; Comment ***@***.***>...