mmrm bioconductor dependancy

Question

mmrm bioconductor dependancy

Closed this issue 6 months ago · 4 comments

When trying to install {mmrm} with {renv} it thinks it needs bioconductor. We think this might be because of the biocViews field in the description. (see image below about renv output)

Answer 1 · 2024-01-25T15:07:26.000Z

@yonicd I think you could track this down to a known renv issue right?

Answer 2 · 2024-01-25T15:10:02.000Z

This issue rstudio/renv#1356 seems to have a similar throughline.

@statasaurus which version of {renv} are you using?

Answer 3 · 2024-01-26T09:17:26.000Z

This is (at least) renv version 1.0.0.

Answer 4 · 2024-01-26T14:28:38.000Z

Could this be just because of https://github.com/openpharma/mmrm/blob/main/DESCRIPTION#L93 ?
If yes, I think we could just remove that line. Seems like it was there from the R package template in the beginning.