bugGNU Octave - Bugs: bug #52595, gnuplot toolkit fails...

 
 

bug #52595: gnuplot toolkit fails "light" demo #11

Submitter:  Dan Sebald <sebald>
Submitted:  Wed 06 Dec 2017 05:08:08 AM UTC
   
 
Category:  Plotting with gnuplot Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 06 Dec 2017 10:10:57 PM UTC, comment #4: 

I applied the patch on stable and pulled it through to default.  I think it is good enough for now so I'll close the report.

Incidentally, looking through the code made me realize there were a lot of instances where choosing the correct quotation symbol, single quote in this case, would really simplify the code because there wouldn't be a whole lot of backslash escapes cluttering everything.  I made a lot of changes in _gnuplot_draw_axes_.m so you are going to want to pull from Mercurial soon.  See http://hg.savannah.gnu.org/hgweb/octave/rev/b1d1229d9e83.

Rik <rik5>
Group administrator
Wed 06 Dec 2017 06:41:24 PM UTC, comment #3: 

How about this minimal patch, simply to solve the error?  This color treatment is so involved in terms of .cdata representation and various combinations I don't want to change too much at the moment.  The demo light 10 and demo light 11 make it obvious that gnuplot toolkit is deficient here.

(file #42584)

Dan Sebald <sebald>
Wed 06 Dec 2017 05:10:34 PM UTC, comment #2: 

OK, I'm looking at this right now and will pull/update to avoid conflicts...

Dan Sebald <sebald>
Wed 06 Dec 2017 05:07:06 PM UTC, comment #1: 

I checked in a change to warn just once per Octave session that the "interp" property for faces is not supported.  See http://hg.savannah.gnu.org/hgweb/octave/rev/8bf013efffa3.  That should reduce the clutter a lot.

Rik <rik5>
Group administrator
Wed 06 Dec 2017 05:08:08 AM UTC, original submission:  

In bug report https://savannah.gnu.org/bugs/?52589 is noted an error with the light function demo 11:

"
I did notice that


graphics_toolkit gnuplot
demo light 11


produced this error


multiplot> splot "-" using 1:2:3:4 title "" with pm3d , "-" binary format='%float64' record=4 using ($1):($2):($3) title "" with linespoints linewidth 0.500000 pointtype -1 dashtype solid pointsize 0.666667 lc rgb "#00 , "-" binary format='%float64' record=4 using ($1):($2):($3) title "" with linespoints linewidth 0.500000 pointtype -1 dashtype solid pointsize 0.666667 lc rgb "

"

What is wrong with the above is the RGB description construction only has the first color component '"#00' rather than the proper '"#rrggbb"'.  I suspect that the warning message before the above error message, i.e.,:


warning: "interp" not supported, using 1st entry of cdata
warning: called from
    __gnuplot_draw_axes__ at line 954 column 21
    __gnuplot_draw_figure__ at line 172 column 17
    __gnuplot_drawnow__ at line 86 column 5
[snip]
warning: "interp" not supported, using 1st entry of cdata
warning: called from
    __gnuplot_draw_axes__ at line 954 column 21
    __gnuplot_draw_figure__ at line 172 column 17
    __gnuplot_drawnow__ at line 86 column 5


is a clue as to why the color string isn't constructed correctly.

Also, let's see if we can reduce all those warnings to just a single warning.

Dan Sebald <sebald>

 

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

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by sebald (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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-12-06 rik5 StatusConfirmed Fixed
        Open/ClosedOpen Closed
    2017-12-06 sebald Attached File- Added octave-gnuplot_tk_interp_edgecolor-djs2017dec06.patch, #42584
    2017-12-06 rik5 StatusNone Confirmed

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code