bugGNU Octave - Bugs: bug #52358, colormap case sensitive

 
 

bug #52358: colormap case sensitive

Submitter:  None
Submitted:  Tue 07 Nov 2017 12:10:53 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Matlab Compatibility
Status:  Fixed Assigned to:  None
Originator Name:  Titus Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * 4.2.1
Operating System:  * Microsoft Windows Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Fri 10 Nov 2017 08:46:32 AM UTC, comment #4: 

Is this fix looks suitable to apply to the stable branch?

Avinoam Kalma <avinoam>
Group Member
Thu 09 Nov 2017 01:10:06 PM UTC, comment #3: 

Yes it took me 3 cset to get this one liner fix right :-). I hope it is actually fixed now.

Pantxo Diribarne <pantxo>
Group Member
Thu 09 Nov 2017 12:47:17 PM UTC, comment #2: 

The fix introduces a regression:


octave: 1> version
ans = 4.3.0+

octave:2> __octave_config_info__.hg_id
ans = 3d96400df713+

octave:3> test colormap
***** error <MAP must be a real-valued N x 3> colormap ([1 i 1])
!!!!! error failed.
Expected <MAP must be a real-valued N x 3>, but got <tolower: not defined for complex matrix>


I guess the tolower(varargin{1}) call should be applied only when varargin{1} is a char.

A.R. Burgers <arb>
Wed 08 Nov 2017 02:10:39 PM UTC, comment #1: 

Thanks for your bug report. I fixed this here:

http://hg.savannah.gnu.org/hgweb/octave/rev/547919e6aec1

Closing report as fixed.

Pantxo Diribarne <pantxo>
Group Member
Tue 07 Nov 2017 12:10:53 PM UTC, original submission:  

In matlab (2016b) both versions are working:

colormap('Copper')

and

colormap('copper')

while with octave only the latter is loading the copper-colors.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by avinoam (Posted a comment)
  • -email is unavailable- added by arb (Posted a comment)
  • -email is unavailable- added by pantxo (Posted a comment)
  • -email is unavailable- added by None (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 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-11-08 pantxo StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code