Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

myassert_size was not declared in this scope #47

Closed
pcarbo opened this issue Jul 26, 2019 · 8 comments
Closed

myassert_size was not declared in this scope #47

pcarbo opened this issue Jul 26, 2019 · 8 comments

Comments

@pcarbo
Copy link

pcarbo commented Jul 26, 2019

I am using R 3.6.1 and gcc 4.8.5 on Linux (CentOS 7.4), and when I try to install the package I get the following error:

clumping.cpp: In function 'Rcpp::List clumping_chr_cached(Rcpp::Environment, arma::sp_mat, const IntegerVector&, const IntegerVector&, const IntegerVector&, const IntegerVector&, const IntegerVector&, const NumericVector&, const NumericVector&, int, double)':
clumping.cpp:101:32: error: 'myassert_size' was not declared in this scope
   myassert_size(spInd.size(), m);

Can you please advise? Thank you.

@pcarbo
Copy link
Author

pcarbo commented Jul 26, 2019

Okay, I was able to install successfully after first installing bigstatsr from GitHub. So it seems that there is a missing version requirement for bigstatsr in DESCRIPTION.

@privefl
Copy link
Owner

privefl commented Jul 27, 2019

The dependency is there: https://github.com/privefl/bigsnpr/blob/master/DESCRIPTION#L25.
I don't know why it is not used, or at least why there is no error saying that the version of {bigstatsr} is not the one required.

Anyway, as you pointed out, using the most recent version of {bigstatsr} is needed for {bigsnpr} and solves this issue.

@privefl privefl closed this as completed Jul 27, 2019
@pcarbo
Copy link
Author

pcarbo commented Jul 27, 2019

You are right. Using the Remotes: field in DESCRIPTION may help when using devtools, but would have to be removed before submitting to CRAN.

@privefl
Copy link
Owner

privefl commented Jul 27, 2019

I already use Remotes: (https://github.com/privefl/bigsnpr/blob/master/DESCRIPTION#L56).

Yet, I think it is only useful for Travis if I remember correctly.

@pcarbo
Copy link
Author

pcarbo commented Jul 27, 2019

In the past it has worked for me, but devtools has changed a lot. One potential workaround is to simply add a step in the README to install bigstatsr from GitHub.

By the way, I am using bigstatsr because it is a dependency of knockoffzoom.

@privefl privefl pinned this issue Aug 26, 2019
@diegoquattrone
Copy link

I have a similar issue when installing bigsnpr:
clumping.cpp:101:3: error: use of undeclared identifier 'myassert_size'
myassert_size(spInd.size(), m);

I tried to update bigstatsr to the last version, however I got another error now when installing bigstatsr:

@diegoquattrone
Copy link

ld: warning: directory not found for option '-L/usr/local/gfortran/lib/gcc/x86_64-apple-darwin15/6.1.0'
ld: warning: directory not found for option '-L/usr/local/gfortran/lib'
ld: library not found for -lgfortran
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [bigstatsr.so] Error 1
ERROR: compilation failed for package ‘bigstatsr’

  • removing ‘/Library/Frameworks/R.framework/Versions/3.6/Resources/library/bigstatsr’
    Error: Failed to install 'bigstatsr' from GitHub:
    (converted from warning) installation of package ‘/var/folders/dk/pg_7gxh10d33v1jqtvb7fvfr0000gn/T//RtmpairyLA/file298d6cff9c9b/bigstatsr_0.9.9.tar.gz’ had non-zero exit status

@privefl
Copy link
Owner

privefl commented Aug 26, 2019

This one is related to {RcppArmadillo}.
Please see #6.

@privefl privefl unpinned this issue Feb 23, 2020
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

No branches or pull requests

3 participants