search

nlmixr2 / rxode2

rxode2
https://nlmixr2.github.io/rxode2/
GNU General Public License v3.0
30 stars 7 forks source link

Windows error with test-npde-mac.R #25

Closed billdenney closed 2 years ago

billdenney commented 2 years ago

I get the following error when I run test-npde-mac.R on Windows 10.

== Testing test-npde-mac.R =====================================================
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 0 ]Assertion failed!

Program: C:\PROGRA~1\R\R-41~1.2\bin\x64\Rterm.exe
File: C:/Users/Bill Denney/Documents/R/win-library/4.1/RcppEigen/include/Eigen/src/Core/PlainObjectBase.h, Line 285

Expression: (!(RowsAtCompileTime!=Dynamic) || (rows==RowsAtCompileTime)) && (!(ColsAtCompileTime!=Dynamic) || (cols==ColsAtCompileTime)) && (!(RowsAtCompileTime==Dynamic && MaxRowsAtCompileTime!=Dynamic) || (rows<=MaxRowsAtCompileTime)) && (!(ColsAtCompileTime==Dynamic && MaxColsAtCompileTime!=Dynamic) || (cols<=MaxColsAtCompileTime)) && rows>=0 && cols>=0 && "Invalid sizes when resizing a matrix or array."

Exited with status 3.
mattfidler commented 2 years ago

This is likely a memory issue, that is not enough memory is able to be allocated in the system, which causes this issue in windows. It was chose because it was a highly memory intensive problem.

billdenney commented 2 years ago

The system I was using has 16GB RAM (I think-- I'm not on it right now). That said, I've had memory exhaustion due to 32-bit processes being used when they should have been 64-bit.

Is there a way that you can think of to give a clear "ran out of memory" error? That seems like it would be a preferred way to close this. If not, we could just skip the test on Windows.

mattfidler commented 2 years ago

Unfortunately, there is no easy way. I have attempted many times.

On Sun, Jan 23, 2022, 6:59 PM Bill Denney @.***> wrote:

The system I was using has 16GB RAM (I think-- I'm not on it right now). That said, I've had memory exhaustion due to 32-bit processes being used when they should have been 64-bit.

Is there a way that you can think of to give a clear "ran out of memory" error? That seems like it would be a preferred way to close this. If not, we could just skip the test on Windows.

— Reply to this email directly, view it on GitHub https://github.com/nlmixr2/rxode2/issues/25#issuecomment-1019617873, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD5VWVLDT6M43FFYOA6S2DUXSP75ANCNFSM5MRBLQPA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you commented.Message ID: @.***>

billdenney commented 2 years ago

This test is now skipped when there is not enough memory on Windows. That's likely sufficient for now.