bugGNU Octave - Bugs: bug #52589, case of multiplot working in 4.0,...

 
 

bug #52589: case of multiplot working in 4.0, but not in 4.2

Submitted by:  Francesco Potortì <pot>
Submitted on:  Tue 05 Dec 2017 03:26:20 PM UTC  
 
Category: Plotting with gnuplotSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Regression
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: 4.2.1Operating System: GNU/Linux

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)

Wed 06 Dec 2017 05:32:59 PM UTC, comment #13:

Yes, I agree that the bug should be closed. I think that the problem with bitmap files is a different issue, and I filed bug #52599

Thanks

Francesco Potortì <pot>
Wed 06 Dec 2017 04:55:29 PM UTC, comment #12:

@Dan: I changed an instance where you had used '%' as a comment character to "##". I checked it in on the stable branch here (http://hg.savannah.gnu.org/hgweb/octave/rev/9cba59ee9564).

I'm going to close this bug report since the issue raised, and the test case, now pass. If there is a different problem with PNG images and saturated colors then please open up a new report with a new example for that.

Rik <rik5>
Project Administrator
Wed 06 Dec 2017 11:21:15 AM UTC, comment #11:

Thank you, it works for the bug demo. I tried it for my complete work and still I see a problem with an included PNG image using only saturated colours. Going to investigate it and reporting here or possibly in a new bug report.

I attach the result of my complete case for gnuplot end fltk, using print() in PDF without any options

(file #42578, file #42579)

Francesco Potortì <pot>
Wed 06 Dec 2017 05:18:40 AM UTC, comment #10:

OK, thanks for reviewing. I've added a bug report for the noted light demo:

https://savannah.gnu.org/bugs/index.php?52595

Dan Sebald <sebald>
Wed 06 Dec 2017 04:45:36 AM UTC, comment #9:

This looks good. I tried the patch on the current stable branch and it solves the problem reported here. I also ran 'rundemos draw' to see if anythig had been upset by the patch, but everything seemed alright.

I did notice that

produced this error

This error is produced regardless of whether the patch is applied so it is a different problem.

Rik <rik5>
Project Administrator
Tue 05 Dec 2017 11:14:33 PM UTC, comment #8:

I ran drawnow() and the obvious problem is the following:

That clearly makes no sense: can't continue a line which is a complete command. I suppose the mixing of image and non-image data wasn't fully exercised after some things having been changed regarding images. I can't recall or deduce why the image should be drawn in it's own plot command while the other data is separate. Probably has something to do with the fact that gnuplot won't draw the background image in the right order for Qt terminal. Whatever the case, I'm attaching a changeset that moves the "unset obj 1" prior to all plot commands, gets rid of the semicolon for image plots, and generally simplifies the code.

Consider this an alpha patch that we will try for a few days. "demo image" and "demo plot" work for me. The "bug" demo works, gnuplot and fltk toolkits producing a similar result--but perhaps the layout isn't quite the same as shown in the original attached PNG.

(file #42575)

Dan Sebald <sebald>
Tue 05 Dec 2017 06:22:51 PM UTC, comment #7:

I installed the image package and I get the same problem with 4.2.1 or the development branch. I still suspect it is related to pipes, but I'm going to add the resident expert on gnuplot, Dan Sebald, to the CC list.

I tried using

and then running bug() but Octave did not stop when gnuplot encountered an error. Most likely will need to use drawnow() to dump the commands Octave is issuing to a file and then try reading those in to gnuplot to verify whether it is Octave or an issue with pipes.

Rik <rik5>
Project Administrator
Tue 05 Dec 2017 05:05:35 PM UTC, comment #6:

Pipe issues were the first thing that came to mind, but then I excluded it because it has always reliably worked on 4.0 and never works on 4.2 here. Ok, I know, that means little...

Anyway, the test case involves two bitmap images. I just tried reducing their resolution to a very low value (8x6 pixels): the problem changes somehow but does not go away.

Francesco Potortì <pot>
Tue 05 Dec 2017 04:47:44 PM UTC, comment #5:

There have been a lot of changes on the development branch with respect to gnuplot. If it works there occasionally then I'm inclined to say the problem of syntax has probably been fixed.

However, there is a deeper issue which is that Octave communicates with gnuplot over a pipe. When there is a lot of data to be communicated, such as an image with one point per pixel, the pipe can get out of sync and then the errors reported are pretty random. This has the feel of that sort of issue.

Rik <rik5>
Project Administrator
Tue 05 Dec 2017 04:31:10 PM UTC, comment #4:

It never works for me on a Debian testing system.

Neither from the command line nor from the GUI version.

Francesco Potortì <pot>
Tue 05 Dec 2017 04:24:20 PM UTC, comment #3:

I get it to work sometimes. see the attached file.
hgid is bffbc95fa72b

But I also saw the errors reported here, when I ran a newer version of octave with ./run-octave.

(file #42572)

Doug Stewart <dastew>
Tue 05 Dec 2017 04:11:34 PM UTC, comment #2:

I just spent about one hour to reduce the test case before sending the report.

Removing the dependence from the image package would require some more work on my part. I can do that if necessary, but I think it would be much faster if you could install the image package, is that a serious problem?

Francesco Potortì <pot>
Tue 05 Dec 2017 03:54:58 PM UTC, comment #1:

I couldn't test because the image package was also required. Can you extract a simpler example that illustrates the problem?

Rik <rik5>
Project Administrator
Tue 05 Dec 2017 03:26:20 PM UTC, original submission:

The attached program works under 4.0, but fails under 4.2.

gnuplot errors out with:
and then spits out lot of errors, all starting with the multiplot> prompt.

To run it, unpack the zip file in a directory and run bug.m

Francesco Potortì <pot>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #42578:  score.pdf added by pot (74KiB - application/pdf - first file uses fltk, second one usees gnuplot)
file #42579:  score.pdf added by pot (51KiB - application/pdf - first file uses fltk, second one usees gnuplot)
file #42571:  bug.zip added by pot (35KiB - application/zip)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by sebald (Updated the item)
  • -unavailable- added by rik5
  • -unavailable- added by dastew (Updated the item)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by pot (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
    Wed 06 Dec 2017 04:55:29 PM UTCrik5StatusPatch Reviewed=>Fixed
      Open/ClosedOpen=>Closed
    Wed 06 Dec 2017 11:21:15 AM UTCpotAttached File-=>Added score.pdf, #42578
      Attached File-=>Added score.pdf, #42579
    Wed 06 Dec 2017 04:45:36 AM UTCrik5StatusNeed Info=>Patch Reviewed
    Tue 05 Dec 2017 11:14:33 PM UTCsebaldAttached File-=>Added octave-gnuplot_tk_image_nonimage-djs2017dec05.patch, #42575
    Tue 05 Dec 2017 06:22:51 PM UTCrik5Carbon-Copy-=>Added sebald
    Tue 05 Dec 2017 04:24:20 PM UTCdastewAttached File-=>Added Screenshot from 2017-12-05 11-18-53.png, #42572
    Tue 05 Dec 2017 03:54:58 PM UTCrik5StatusNone=>Need Info
    Tue 05 Dec 2017 03:26:20 PM UTCpotAttached File-=>Added bug.zip, #42571

    Back to the top


    Powered by Savane 3.1-cleanup1