Re: bug in linkaxes

[avlas]

El divendres, 27 d’abril de 2018, a les 13:57:51 EDT, Rik va escriure:

  On 04/27/2018 10:52 AM, avlas wrote:
  > El divendres, 27 d’abril de 2018, a les 13:42:10 EDT, Rik va escriure:
  >
  >   On 04/27/2018 10:25 AM, avlas wrote:
  >   > El divendres, 27 d’abril de 2018, a les 13:10:51 EDT, avlas va escriure:
  >   >
  >   >   El divendres, 27 d’abril de 2018, a les 12:59:30 EDT, Rik va escriure:
  >   >   
  >   >     On 04/27/2018 07:33 AM, address@hidden wrote:
  >   >     > Subject:
  >   >     > bug in linkaxes ?
  >   >     > From:
  >   >     > avlas <address@hidden>
  >   >     > Date:
  >   >     > 04/27/2018 07:23 AM
  >   >     >
  >   >     > To:
  >   >     > octave-maintainers <address@hidden>
  >   >     >
  >   >     > List-Post:
  >   >     > <mailto:address@hidden>
  >   >     > Content-Transfer-Encoding:
  >   >     > 7Bit
  >   >     > Precedence:
  >   >     > list
  >   >     > MIME-Version:
  >   >     > 1.0
  >   >     > Message-ID:
  >   >     > <address@hidden>
  >   >     > Content-Type:
  >   >     > text/plain; charset="us-ascii"
  >   >     > Message:
  >   >     > 4
  >   >     >
  >   >     >
  >   >     > I think there is a bug in linkaxes, at least in my Octave version 
(flatpak package of Octave 4.3.91).
  >   >     >
  >   >     > Before submitting a bug report about this, could somebody please 
test the following code snippet? Thanks!
  >   >     >
  >   >     >   % these examples work fine
  >   >     >   % for ii = 1:3; ha(ii) = subplot(3,1,ii); axes(ha(ii)); 
plot(randn(10,ii)); end
  >   >     >   % for ii = 1:3; ha(ii) = subplot(2,2,ii); axes(ha(ii)); 
plot(randn(10,ii)); end
  >   >     >   
  >   >     >   % these examples do NOT work
  >   >     >   for ii = 1:4; ha(ii) = subplot(4,1,ii); axes(ha(ii)); 
plot(randn(10,ii)); end
  >   >     >   % for ii = 1:4; ha(ii) = subplot(2,2,ii); axes(ha(ii)); 
plot(randn(10,ii)); end
  >   >     >   
  >   >     >   linkaxes(ha, 'x')
  >   >     >
  >   >     > This is the error I'm finding:
  >   >     >
  >   >     >   error: horizontal dimensions mismatch (1x1 vs 2x1)
  >   >     >   error: called from
  >   >     >       linkprop at line 84 column 7
  >   >     >       linkaxes at line 83 column 13
  >   >     >
  >   >     > ...
  >   >     > a.
  >   >     This was a simple error in using a comma, ',', for horizontal 
concatenation
  >   >     when a semicolon, ';', was desired for vertical concatenation.  I 
fixed it
  >   >     on the stable branch here
  >   >     http://hg.savannah.gnu.org/hgweb/octave/rev/2540fde5f07b.
  >   >     
  >   >     --Rik
  >   >     
  >   >   Thank you
  >   >   ....
  >   >   a.
  >   >   
  >   > Unfortunately the commit didn't fix the issue. Could you please try to 
reproduce?
  >   > ....
  >   > a.
  >   Works for me with the test code that was uncommented in your e-mail.
  >   
  >   for ii = 1:4; ha(ii) = subplot(4,1,ii); axes(ha(ii)); plot(randn(10,ii)); 
end
  >   linkaxes (ha, 'x')
  >   
  >   --Rik
  >   
  > Strangely enough, it works for me too now. I updated the linkprop file on 
my own after running it, and somehow the change was not taken into account 
until I restarted Octave, which seems weird to me. I'd like to know if this is 
something general or related to flatpak :/
  Probably flatpak.  Octave looks at the timestamp of an m-file each time it
  is used to decide whether it needs to be re-compiled internally.  Unless
  there was some weirdness with the file system, it should have seen the
  updated file that was copied in to the search path and used it.
  
  --Rik
  
I see. Thank you for the information. It's good to know to avoid misleading 
results when changing code, although changing Octave internal functions is not 
sth frequently needed.

....
a.