(
Jump to the original submission)
Tue 15 Oct 2013 06:53:27 PM UTC, comment #8:
I have finally pushed my fix:
http://hg.savannah.gnu.org/hgweb/octave/rev/0b7f5c56f853
This should be part of the upcoming release.
|
Wed 17 Jul 2013 05:20:32 PM UTC, comment #7:
At the time I wrote my original comments, I was going off memory without reading the actual source code. Now that I look at it again, I realise that a slightly more sophisticated fix is required, first see if m will fit into an octave_idx_type, and if it doesn't then force a short shuffle.
It's a bit more than a one-liner right now, and at the moment I can't dedicate any time to this possibly easy problem.
|
Wed 17 Jul 2013 03:43:41 PM UTC, comment #6:
Jordi, can you apply this one-liner fix of yours or is there more testing to be done?
|
Wed 03 Jul 2013 02:01:50 PM UTC, comment #5:
Perfect.
Marco
|
Wed 03 Jul 2013 01:26:11 PM UTC, comment #4:
No, I meant double(n) > ..., rather. The problem is that if the full size of the vector is too big, then there's no hope that the "long" shuffle can work, because then line 1149 fails:
http://hg.savannah.gnu.org/hgweb/octave/file/76bd90f6ba65/libinterp/corefcn/rand.cc#l1147
In this case, we must do a short shuffle.
|
Wed 03 Jul 2013 08:25:28 AM UTC, comment #3:
Probably you mean
Marco
|
Tue 02 Jul 2013 07:40:12 PM UTC, comment #2:
I had some reason for capping the size of M. My reasoning was that if M was big enough, you might as well just allocate the full vector for speed. Perhaps a more sophisticated approach would be to do
?
|
Tue 02 Jul 2013 02:02:49 PM UTC, comment #1:
The workaround is to change the "quick and dirty" test in rand.cc into
Marco
|
Mon 01 Jul 2013 07:28:51 AM UTC, original submission:
Dear all,
the following
gives me the "memory exhausted or requested size too large for range of Octave's index type" error. I think it should work since:
1) N < intmax
2) M < N/5 and so O(M) memory should be used (according to help)
3) Matlab works with such an example
Best regards,
Marco
|
(Note: upload size limit is set to 16384 kB, after insertion of the required
escape characters.)
Attach File(s):
Comment:
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
Follow 4 latest changes.