Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Dirk Eddelbuettel
On 28 January 2016 at 21:47, Paul Johnson wrote: | Thanks. | | On Thu, Jan 28, 2016 at 2:42 PM, Dirk Eddelbuettel wrote: | > | > Paul, | > | > I can reproduce the segfault on Ubuntu 15.10, "everything current". | > Definitely a valid bug report, though a 30mb xlsx may not qualify as | > minimal.

Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Kevin Ushey
When I add some debug printing to the associated subscripting line (https://github.com/awalker89/openxlsx/blob/b92bb3acdd6ea759be928c298c6faeef2f26fa3e/src/cppFunctions.cpp#L2608), I see: colNumbers.size(): 98,03,150 charCols.size(): 95,94,546 It looks to me like the package is erroneously

Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Dirk Eddelbuettel
On 29 January 2016 at 11:27, Kevin Ushey wrote: | When I add some debug printing to the associated subscripting line | (https://github.com/awalker89/openxlsx/blob/b92bb3acdd6ea759be928c298c6faeef2f26fa3e/src/cppFunctions.cpp#L2608), | I see: | |colNumbers.size(): 98,03,150 |charCols.size

Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Qiang Kou
Hi, Kevin, I was also trying to track this down yesterday. >From the debugging info below, *indices_n* is not equal to length of *indices*, which I don't quite understand. Program received signal SIGSEGV, Segmentation fault. 0x72ed5c4e in Rcpp::SubsetProxy<13, Rcpp::PreserveStorage, 13,

Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Qiang Kou
Hi, Paul, can you try my fork of Rcpp? You can install it by the line below: devtools::install_github("thirdwing/Rcpp", ref = "subsetter") This fixed the segfault on my Ubuntu machine. The difference can be found from [1]. In *subsetter*, if an IntegerVector passed in, we will try to reuse it.

Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Dirk Eddelbuettel
On 29 January 2016 at 17:55, Qiang Kou wrote: | Hi, Paul, can you try my fork of Rcpp? You can install it by the line below: | | devtools::install_github("thirdwing/Rcpp", ref = "subsetter") | | This fixed the segfault on my Ubuntu machine. Yay. Nice work! | The difference can be found from [

Re: [Rcpp-devel] Possible regression in R-3.2.3 or Rcpp 0.12.3

2016-01-29 Thread Kevin Ushey
Hmm, I have one thought: we try to re-use the indices from an IntegerVector, but the type here is actually a sugar type: Rcpp::SubsetProxy<13, Rcpp::PreserveStorage, 13, true, Rcpp::sugar::Minus_Vector_Primitive<13, true, Rcpp::Vector<13, Rcpp::PreserveStorage> > >::get_vec (this=) Ie