search

rvlenth / emmeans

Estimated marginal means
https://rvlenth.github.io/emmeans/
340 stars 30 forks source link

Building from github fails on Windows #497

Closed Generalized closed 4 days ago

Generalized commented 5 days ago

Hi,

I use the command remotes::install_github("rvlenth/emmeans", dependencies = TRUE, build_opts = "")

It always worked, until now. I wanted to update the version that pools the degrees of freedom from imputed data. I noticed you mentioned some error in the code (*2 rather than ^2) and I'm afraid if my current version has this error...

But now it fails completely

Enter one or more numbers, or an empty line to skip updates: 
── R CMD build ────────────────────────────────────────────────────────────────────────────────────────────────────────────
✔  checking for file 'C:\Users\.........\Temp\RtmpaGmsoH\remotes3a20159a5b54\rvlenth-emmeans-ce2cddf/DESCRIPTION' ... 
─  preparing 'emmeans': (433ms)
✔  checking DESCRIPTION meta-information ... 
─  checking for LF line-endings in source and make files and shell scripts (419ms)
─  checking for empty or unneeded directories
─  building 'emmeans_1.10.3.tar.gz'

    ERROR
   cannot change to directory ''
Error: Failed to install 'emmeans' from GitHub:
! System command 'Rcmd.exe' failed

I use R 4.2.0

OK, it works when the parameter build_opts is removed, i.e. this work: remotes::install_github("rvlenth/emmeans")

rvlenth commented 5 days ago

Use install.packages("emmeans"). It's the same version right now.

Generalized commented 5 days ago

Thank you. It also works when the parameter build_opts is removed, i.e. this work: remotes::install_github("rvlenth/emmeans")

And thank God I updated it, because I realized that the DFs are different from what I had previously working with imputed data. Now each row has different DF, which shows that the pooling is actually done through Rubin's rules. Now I have the chance to update all analyses before sending my work.

rvlenth commented 4 days ago

I get the same error. I changed the instructions to omit build_opts and add build_vignettes = TRUE with a note that this argument may be omitted. It will be in the next push.