[julia-users] Re: hist on matrices

2014-04-08 Thread Ivar Nesje
This should be fixed with https://github.com/JuliaLang/julia/pull/6462

kl. 07:39:05 UTC+2 tirsdag 8. april 2014 skrev Ivar Nesje følgende:

 Thanks for the report.

 I found the bug in 
 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl#L470, 
 because it tries to calculate length(edg) - 1, but actually calculates 
 length(edg - 1), which is kind of useless.

 I'll work on a patch (and maybe add a test).

 Ivar

 kl. 01:26:43 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 If I recall correctly, it used to be that for a matrix M, hist(M)[2] 
 would return a matrix whose ith column would be the histogram of the ith 
 column of M. I know 'hist' was recently updated, and now whenever I call it 
 on a matrix, I get the following error:

 julia x = rand(10,10);

 julia hist(x)
 ERROR: Incorrect size of H.
  in hist! at statistics.jl:460
  in hist at statistics.jl:470
  in hist at statistics.jl:472

 I get this even if I provide the number of bins or bin edges. I believe H 
 is where hist! is writing the column-wise histogram, but I'm not sure why 
 it's always of the wrong size. I may be misusing the updated 'hist', but I 
 can't figure out how from the code and documentation. Thanks in advance for 
 any help.

 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl

 julia versioninfo()
 Julia Version 0.3.0-prerelease+2503
 Commit 2990c29 (2014-04-06 04:30 UTC)
 Platform Info:
   System: Darwin (x86_64-apple-darwin13.1.0)
   CPU: Intel(R) Core(TM) i7 CPU   M 620  @ 2.67GHz
   WORD_SIZE: 64
   BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
   LAPACK: libopenblas
   LIBM: libopenlibm



[julia-users] Re: hist on matrices

2014-04-08 Thread Stephen Voinea
Thanks again!

On Tuesday, April 8, 2014 2:00:50 AM UTC-4, Ivar Nesje wrote:

 This should be fixed with https://github.com/JuliaLang/julia/pull/6462

 kl. 07:39:05 UTC+2 tirsdag 8. april 2014 skrev Ivar Nesje følgende:

 Thanks for the report.

 I found the bug in 
 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl#L470, 
 because it tries to calculate length(edg) - 1, but actually calculates 
 length(edg - 1), which is kind of useless.

 I'll work on a patch (and maybe add a test).

 Ivar

 kl. 01:26:43 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 If I recall correctly, it used to be that for a matrix M, hist(M)[2] 
 would return a matrix whose ith column would be the histogram of the ith 
 column of M. I know 'hist' was recently updated, and now whenever I call it 
 on a matrix, I get the following error:

 julia x = rand(10,10);

 julia hist(x)
 ERROR: Incorrect size of H.
  in hist! at statistics.jl:460
  in hist at statistics.jl:470
  in hist at statistics.jl:472

 I get this even if I provide the number of bins or bin edges. I believe 
 H is where hist! is writing the column-wise histogram, but I'm not sure why 
 it's always of the wrong size. I may be misusing the updated 'hist', but I 
 can't figure out how from the code and documentation. Thanks in advance for 
 any help.

 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl

 julia versioninfo()
 Julia Version 0.3.0-prerelease+2503
 Commit 2990c29 (2014-04-06 04:30 UTC)
 Platform Info:
   System: Darwin (x86_64-apple-darwin13.1.0)
   CPU: Intel(R) Core(TM) i7 CPU   M 620  @ 2.67GHz
   WORD_SIZE: 64
   BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
   LAPACK: libopenblas
   LIBM: libopenlibm



[julia-users] Re: hist on matrices

2014-04-08 Thread Stephen Voinea
Happy to hear I didn't just point out a bunch of correctly working code 
that I didn't understand. Take care. 

On Tuesday, April 8, 2014 2:36:23 AM UTC-4, Ivar Nesje wrote:

 I was also confused by  sub(H(H, :, j), so I just removed it. 

 kl. 08:12:04 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 Thanks again!

 On Tuesday, April 8, 2014 2:00:50 AM UTC-4, Ivar Nesje wrote:

 This should be fixed with https://github.com/JuliaLang/julia/pull/6462

 kl. 07:39:05 UTC+2 tirsdag 8. april 2014 skrev Ivar Nesje følgende:

 Thanks for the report.

 I found the bug in 
 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl#L470, 
 because it tries to calculate length(edg) - 1, but actually calculates 
 length(edg - 1), which is kind of useless.

 I'll work on a patch (and maybe add a test).

 Ivar

 kl. 01:26:43 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 If I recall correctly, it used to be that for a matrix M, hist(M)[2] 
 would return a matrix whose ith column would be the histogram of the ith 
 column of M. I know 'hist' was recently updated, and now whenever I call 
 it 
 on a matrix, I get the following error:

 julia x = rand(10,10);

 julia hist(x)
 ERROR: Incorrect size of H.
  in hist! at statistics.jl:460
  in hist at statistics.jl:470
  in hist at statistics.jl:472

 I get this even if I provide the number of bins or bin edges. I 
 believe H is where hist! is writing the column-wise histogram, but I'm 
 not 
 sure why it's always of the wrong size. I may be misusing the updated 
 'hist', but I can't figure out how from the code and documentation. 
 Thanks 
 in advance for any help.

 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl

 julia versioninfo()
 Julia Version 0.3.0-prerelease+2503
 Commit 2990c29 (2014-04-06 04:30 UTC)
 Platform Info:
   System: Darwin (x86_64-apple-darwin13.1.0)
   CPU: Intel(R) Core(TM) i7 CPU   M 620  @ 2.67GHz
   WORD_SIZE: 64
   BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
   LAPACK: libopenblas
   LIBM: libopenlibm



[julia-users] Re: hist on matrices

2014-04-08 Thread Stephen Voinea
Already done and tested. Works great. Thanks for making that happen to 
quickly, Ivar. 

On Tuesday, April 8, 2014 2:15:57 PM UTC-4, Ivar Nesje wrote:

 Now the fix has been merged, so now you just have to pull and make to get 
 a working version (or wait for the next nightly release).

 Ivar

 kl. 19:50:34 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 Happy to hear I didn't just point out a bunch of correctly working code 
 that I didn't understand. Take care. 

 On Tuesday, April 8, 2014 2:36:23 AM UTC-4, Ivar Nesje wrote:

 I was also confused by  sub(H(H, :, j), so I just removed it. 

 kl. 08:12:04 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 Thanks again!

 On Tuesday, April 8, 2014 2:00:50 AM UTC-4, Ivar Nesje wrote:

 This should be fixed with https://github.com/JuliaLang/julia/pull/6462

 kl. 07:39:05 UTC+2 tirsdag 8. april 2014 skrev Ivar Nesje følgende:

 Thanks for the report.

 I found the bug in 
 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl#L470, 
 because it tries to calculate length(edg) - 1, but actually calculates 
 length(edg - 1), which is kind of useless.

 I'll work on a patch (and maybe add a test).

 Ivar

 kl. 01:26:43 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea 
 følgende:

 If I recall correctly, it used to be that for a matrix M, hist(M)[2] 
 would return a matrix whose ith column would be the histogram of the 
 ith 
 column of M. I know 'hist' was recently updated, and now whenever I 
 call it 
 on a matrix, I get the following error:

 julia x = rand(10,10);

 julia hist(x)
 ERROR: Incorrect size of H.
  in hist! at statistics.jl:460
  in hist at statistics.jl:470
  in hist at statistics.jl:472

 I get this even if I provide the number of bins or bin edges. I 
 believe H is where hist! is writing the column-wise histogram, but I'm 
 not 
 sure why it's always of the wrong size. I may be misusing the updated 
 'hist', but I can't figure out how from the code and documentation. 
 Thanks 
 in advance for any help.

 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl

 julia versioninfo()
 Julia Version 0.3.0-prerelease+2503
 Commit 2990c29 (2014-04-06 04:30 UTC)
 Platform Info:
   System: Darwin (x86_64-apple-darwin13.1.0)
   CPU: Intel(R) Core(TM) i7 CPU   M 620  @ 2.67GHz
   WORD_SIZE: 64
   BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
   LAPACK: libopenblas
   LIBM: libopenlibm



[julia-users] Re: hist on matrices

2014-04-07 Thread Ivar Nesje
Thanks for the report.

I found the bug 
in https://github.com/JuliaLang/julia/blob/master/base/statistics.jl#L470, 
because it tries to calculate length(edg) - 1, but actually calculates 
length(edg - 1), which is kind of useless.

I'll work on a patch (and maybe add a test).

Ivar

kl. 01:26:43 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 If I recall correctly, it used to be that for a matrix M, hist(M)[2] would 
 return a matrix whose ith column would be the histogram of the ith column 
 of M. I know 'hist' was recently updated, and now whenever I call it on a 
 matrix, I get the following error:

 julia x = rand(10,10);

 julia hist(x)
 ERROR: Incorrect size of H.
  in hist! at statistics.jl:460
  in hist at statistics.jl:470
  in hist at statistics.jl:472

 I get this even if I provide the number of bins or bin edges. I believe H 
 is where hist! is writing the column-wise histogram, but I'm not sure why 
 it's always of the wrong size. I may be misusing the updated 'hist', but I 
 can't figure out how from the code and documentation. Thanks in advance for 
 any help.

 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl

 julia versioninfo()
 Julia Version 0.3.0-prerelease+2503
 Commit 2990c29 (2014-04-06 04:30 UTC)
 Platform Info:
   System: Darwin (x86_64-apple-darwin13.1.0)
   CPU: Intel(R) Core(TM) i7 CPU   M 620  @ 2.67GHz
   WORD_SIZE: 64
   BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
   LAPACK: libopenblas
   LIBM: libopenlibm



[julia-users] Re: hist on matrices

2014-04-07 Thread Stephen Voinea
Ah, that's it. Thanks for looking into it. 

On another note (but in the same code neighborhood), what's the meaning of 

sub(H(H, :, j)

in line 465 of statistics.jl? I'm confused by that first 'H', as I thought 
it would imply a function call.

On Tuesday, April 8, 2014 1:39:05 AM UTC-4, Ivar Nesje wrote:

 Thanks for the report.

 I found the bug in 
 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl#L470, 
 because it tries to calculate length(edg) - 1, but actually calculates 
 length(edg - 1), which is kind of useless.

 I'll work on a patch (and maybe add a test).

 Ivar

 kl. 01:26:43 UTC+2 tirsdag 8. april 2014 skrev Stephen Voinea følgende:

 If I recall correctly, it used to be that for a matrix M, hist(M)[2] 
 would return a matrix whose ith column would be the histogram of the ith 
 column of M. I know 'hist' was recently updated, and now whenever I call it 
 on a matrix, I get the following error:

 julia x = rand(10,10);

 julia hist(x)
 ERROR: Incorrect size of H.
  in hist! at statistics.jl:460
  in hist at statistics.jl:470
  in hist at statistics.jl:472

 I get this even if I provide the number of bins or bin edges. I believe H 
 is where hist! is writing the column-wise histogram, but I'm not sure why 
 it's always of the wrong size. I may be misusing the updated 'hist', but I 
 can't figure out how from the code and documentation. Thanks in advance for 
 any help.

 https://github.com/JuliaLang/julia/blob/master/base/statistics.jl

 julia versioninfo()
 Julia Version 0.3.0-prerelease+2503
 Commit 2990c29 (2014-04-06 04:30 UTC)
 Platform Info:
   System: Darwin (x86_64-apple-darwin13.1.0)
   CPU: Intel(R) Core(TM) i7 CPU   M 620  @ 2.67GHz
   WORD_SIZE: 64
   BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY)
   LAPACK: libopenblas
   LIBM: libopenlibm