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

 
 

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

Submitted by:  Dan Sebald <sebald>
Submitted on:  Wed 06 Dec 2017 05:08:08 AM UTC  
 
Category: Plotting with gnuplotSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Unexpected Error
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.

 

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>
Project 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>
Project 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

produced this error

"

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.,:

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 File(s):
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 06 Dec 2017 10:10:57 PM UTCrik5StatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
    Wed 06 Dec 2017 06:41:24 PM UTCsebaldAttached File-=>Added octave-gnuplot_tk_interp_edgecolor-djs2017dec06.patch, #42584
    Wed 06 Dec 2017 05:07:06 PM UTCrik5StatusNone=>Confirmed

    Back to the top


    Powered by Savane 3.1-cleanup1