Skip to content

Update mccrackendata.R#5

Open
franz-maikaefer wants to merge 1 commit into
grayclhn:masterfrom
franz-maikaefer:mismatched_dimnames_patch
Open

Update mccrackendata.R#5
franz-maikaefer wants to merge 1 commit into
grayclhn:masterfrom
franz-maikaefer:mismatched_dimnames_patch

Conversation

@franz-maikaefer
Copy link
Copy Markdown

Match dimnames to array dimensions (dim).

In this way, the lazydata step in library build doesn't blow up with an error:


$ R CMD INSTALL oosanalysis-R-library

  • installing to library 'C:/Users/userxxx/Documents/R/win-library/4.0'
  • installing source package 'oosanalysis' ...
    ** using staged installation
    ** R
    ** data
    *** moving datasets to lazyload DB
    Error in array(dim = c(12, 3, 10), dimnames = c("pi", "confidence", "k2"), :
    'dimnames' must be a list
    ERROR: lazydata failed for package 'oosanalysis'
  • removing 'C:/Users/userxxx/Documents/R/win-library/4.0/oosanalysis'
  • restoring previous 'C:/Users/userxxx/Documents/R/win-library/4.0/oosanalysis'

Match dimnames to array dimensions (dim).
@Jak2029
Copy link
Copy Markdown

Jak2029 commented Dec 8, 2021

Hello franz-maikäfer I keep getting the same ERROR. Could you please let me know how I can restore the previous one (code?). Because the installation remotes::install_github("grayclhn/oosanalysis-R-library") did not even work.

@franz-maikaefer
Copy link
Copy Markdown
Author

I didn't alter the code in your repository, but in my fork (https://github.com/franz-maikaefer/oosanalysis-R-library): the changes i proposed there were just to make it possible to build your library, as i don't want to diverge from your repo, but just contribute to it.

Indeed, when you try to install using remotes::install_github("grayclhn/oosanalysis-R-library") you're using your code as it was - hence the error that motivated my patch (pending your approval), in my fork.

Try remotes::install_github("franz-maikaefer/oosanalysis-R-library"). It should install.

And if you judge that the proposed changes are welcomed, you can accept it and other users of your library will be able to install it using remotes::install_github("grayclhn/oosanalysis-R-library").

@moatazbellahfarid
Copy link
Copy Markdown

moatazbellahfarid commented Dec 19, 2021

Hi Franz-maikaefer, I keep on getting the same error, can you please tell me how to correct it and make it work?

  • installing source package 'oosanalysis' ...
    ** using staged installation
    ** R
    ** data
    *** moving datasets to lazyload DB
    Error in array(dim = c(12, 3, 10), dimnames = c("pi", "confidence", "k2"), :
    'dimnames' must be a list
    ERROR: lazydata failed for package 'oosanalysis'
  • removing 'C:/Users/Documents/R/win-library/3.6/oosanalysis'
  • restoring previous 'C:/Users/Documents/R/win-library/3.6/oosanalysis'

@sohansust
Copy link
Copy Markdown

sohansust commented Jun 23, 2022

Hi everyone,
Any update with the above error?
I am getting same error with both packages.

@SchlabberBug
Copy link
Copy Markdown

Hi everyone,
I found a workaround by installing the branch with franz-maikaefers commit directly. Just use:

install.packages("githubinstall")
library(githubinstall)
gh_install_packages("franz-maikaefer/oosanalysis-R-library", ref = "9b4251b")
library(oosanalysis)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants