bugGNU Octave - Bugs: bug #36193, scatter(X,Y,..,C) fails

 
 

bug #36193: scatter(X,Y,..,C) fails

Submitted by:  S T Sarkar <bobatnet>
Submitted on:  Sun 15 Apr 2012 02:10:43 PM UTC  
 
Category: PlottingSeverity: 3 - Normal
Priority: 5 - NormalItem Group: None
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Tue 19 Jun 2012 11:03:09 PM UTC, comment #10:

Ok. I've pushed the changest in John's name.

http://hg.savannah.gnu.org/hgweb/octave/rev/566cf544d020

Ben Abbott <bpabbott>
Project Member
Tue 19 Jun 2012 01:26:56 PM UTC, comment #9:

Oh, my apologies Ben. I thought you would push it, since I thought you were the one who knew this part of the code best. Weren't you the one who implemented the prior scatter fix?

Ah, oops, I assigned this to myself. I just undid that.

Jordi Gutiérrez Hermoso <jordigh>
Project Administrator
Tue 19 Jun 2012 12:03:43 PM UTC, comment #8:

Jordi/John

Has the patch been pushed ?

Ben Abbott <bpabbott>
Project Member
Wed 09 May 2012 06:40:44 AM UTC, comment #7:

"columns (c)" is fine, but the mail should end with ".us" :
John Hunt <huntj@gmx.us>

John Hunt <huntj>
Wed 09 May 2012 03:50:17 AM UTC, comment #6:

Hey, John Hunt.

Thanks for the patch. Is it ok if I credit your patch as "John Hunt <huntj@gmx.de>"?

Also, as a matter of style, we prefer to use "columns (c)" instead of "size (c, 2)". I hope you're ok with that instead. If so, I'll push this patch, credited to you.

Jordi Gutiérrez Hermoso <jordigh>
Project Administrator
Sun 06 May 2012 06:59:20 AM UTC, comment #5:

i get the same error as described earlier (error: invalid value for color property "markerfacecolor" ...), both under Ubuntu 12.04 with octave 3.2.4, and under Gentoo with octave 3.6.0, and i can confirm that the patch (file #25652) solves the issue.

hoijui

Anonymous
Wed 25 Apr 2012 07:22:55 PM UTC, comment #4:

in answer to comment #2 :
size(pm6)
ans =

153 5

it is available in the "data" file #25650 attached.
"load data" worked fine here (octave 3.6.1, linux)

John Hunt <huntj>
Wed 25 Apr 2012 06:12:00 PM UTC, comment #3:

The patch solves the problem to do the following plot with gnuplot the same way as with fltk

[x,y,z] = meshgrid(-10:1:10,-5:1:5,-8:2:8);
% this creates x,y and z 3-D arrays, x contains -10:1:10 replicated
length(-5:1:5)xlength(-8:2:8) times, y contains -5:1;5 replicated .....
c = x.^2+y.^2+z.^2;
% c is now our function of 3 variables
figure
scatter3(x(:),y(:),z(:),5,c(:),'filled')

before applying it it gave the same error as in the first post.

- Martin Helm

Anonymous
Wed 25 Apr 2012 05:50:01 PM UTC, comment #2:

What is pm6? Specifically, what size is it?

Jordi Gutiérrez Hermoso <jordigh>
Project Administrator
Sun 15 Apr 2012 05:00:01 PM UTC, comment #1:

Same bug under linux

This seems related to the "gnuplot_hack" line 271 of _scatter_.m
It does not work when c is a vector.
changing this line to
removes the error in this case.

But then the the following fails (it should plot black points)

finally changing this line 271 of _scatter_.m to

seems to work in both cases.
Patch attached.

(file #25652)

John Hunt <huntj>
Sun 15 Apr 2012 02:10:43 PM UTC, original submission:

The following plot, involving a scatter plot, fails with the error:

error: invalid value for color property "markerfacecolor"
error: called from:
error: D:\Octave\Octave-3.6.1\share\octave\3.6.1\m\plot\private\__scatter__.m at line 274, column 11
error: D:\Octave\Octave-3.6.1\share\octave\3.6.1\m\plot\private\__scatter__.m at line 211, column 13
error: D:\Octave\Octave-3.6.1\share\octave\3.6.1\m\plot\scatter.m at line 71, column 11

The same command, though when run from MATLAB works without any error.

S T Sarkar <bobatnet>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #25652:  __scatter__.m.patch added by huntj (643B - text/x-patch)
file #25650:  data added by bobatnet (5KiB - application/octet-stream - file "data" contains the matrix pm6)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rik5 (Updated the item)
  • -unavailable- added by bpabbott (Posted a comment)
  • -unavailable- added by jordigh (Posted a comment)
  • -unavailable- added by huntj (Updated the item)
  • -unavailable- added by bobatnet (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 02 Oct 2012 04:37:26 AM UTCrik5StatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 19 Jun 2012 11:03:09 PM UTCbpabbottStatusPatch Submitted=>Ready For Test
      Release3.6.1=>dev
      Operating SystemMicrosoft Windows=>Any
    Tue 19 Jun 2012 01:26:56 PM UTCjordighAssigned tojordigh=>None
    Wed 09 May 2012 03:50:17 AM UTCjordighAssigned toNone=>jordigh
    Wed 09 May 2012 03:50:16 AM UTCjordighStatusNone=>Patch Submitted
    Sun 15 Apr 2012 05:00:01 PM UTChuntjAttached File-=>Added _scatter_.m.patch, #25652
    Sun 15 Apr 2012 02:10:43 PM UTCbobatnetAttached File-=>Added data, #25650

    Back to the top


    Powered by Savane 3.1-cleanup1