search

openpharma / mmrm

Mixed Models for Repeated Measures (MMRM) in R.
https://openpharma.github.io/mmrm/
Other
101 stars 17 forks source link

mmrm bioconductor dependancy #418

Closed statasaurus closed 4 months ago

statasaurus commented 5 months ago

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)

image
danielinteractive commented 5 months ago

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

yonicd commented 5 months ago

This issue https://github.com/rstudio/renv/issues/1356 seems to have a similar throughline.

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

andyofsmeg commented 5 months ago

This is (at least) renv version 1.0.0.

danielinteractive commented 5 months ago

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.