bugGNU Octave - Bugs: bug #56323, Default (c)transpose for classdef...

 
 

bug #56323: Default (c)transpose for classdef classes

Submitter:  Kai Torben Ohlhus <siko1056>
Submitted:  Wed 15 May 2019 12:46:47 PM UTC
   
 
Category:  Classdef Severity:  2 - Minor
Priority:  5 - Normal Item Group:  Matlab Compatibility
Status:  Confirmed Assigned to:  siko1056
Originator Name:  Open/Closed:  * Open
Release:  * 5.1.0 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 20 May 2019 07:52:49 AM UTC, comment #1: 

Partial fix for the old style classes: https://hg.savannah.gnu.org/hgweb/octave/rev/4ae89360ca16.

Kai Torben Ohlhus <siko1056>
Group Member
Wed 15 May 2019 12:46:47 PM UTC, original submission:  

When creating arrays of old style objects, as shown by the Blork class

https://hg.savannah.gnu.org/hgweb/octave/file/15d3f8857326/test/classes/%40Blork

Matlab R2019a is able to (conjugate) transpose them without overloading (c)transpose, what essentially is a modification of the array shape.


x1 = Blork ();
x2 = [x1 x1];
x2';
error: ctranspose method not defined for Blork class
x2.';
error: transpose method not defined for Blork class


The same applies for classdef classes, using the plist_t2 class as example  https://hg.savannah.gnu.org/hgweb/octave/file/15d3f8857326/test/classdef


bla(1) = plist_t2;
bla(2) = plist_t2;
bla'
error: ctranspose method not defined for plist_t2 class
bla.'
error: transpose method not defined for plist_t2 class


The fix for the old style classes is rather simple and avoids slowdown by not using m-files for standard behavior.  Thus I would like to include it into 5.2.0.

Kai Torben Ohlhus <siko1056>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #46912:  class_ctranspose.patch added by siko1056 (4KiB - text/x-patch)

 

Carbon-Copy List
  • -email is unavailable- added by mmuetzel (Updated the item)
  • -email is unavailable- added by rik5 (Updated the item)
  • -email is unavailable- added by siko1056 (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2024-01-23 mmuetzel CategoryInterpreter Classdef
    2024-01-18 rik5 StatusPatch Submitted Confirmed
        SummaryDefault (c)transpose for classdef and old style classes Default (c)transpose for classdef classes
        Dependencies- Depends on bugs #65179
    2024-01-18 rik5 Dependencies- Depends on bugs #44665
    2019-05-15 siko1056 Attached File- Added class_ctranspose.patch, #46912

    Back to the top

    Powered by Savane 3.13-58d6.
    Corresponding source code