bugGNU Octave - Bugs: bug #38914, Gnuplot axis reverse is not...

 
 

bug #38914: Gnuplot axis reverse is not working with gnuplot 5

Submitted by:  Dan Sebald <sebald>
Submitted on:  Tue 07 May 2013 12:20:05 AM UTC  
 
Category: Plotting with gnuplotSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: Patch SubmittedAssigned to: None
Originator Name: Open/Closed: Open
Release: devOperating System: Any

Add a New Comment (Rich MarkupRich 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)

Sun 24 May 2015 05:04:31 AM UTC, comment #11:

Doesn't seem to cause any problems with gnuplot 4.6.6. Should this go on stable or default?

Rik <rik5>
Project Administrator
Sun 24 May 2015 04:55:14 AM UTC, comment #10:

Updated changeset attached...

(file #34085)

Dan Sebald <sebald>
Sun 24 May 2015 12:48:13 AM UTC, comment #9:

@Dan: I'm running gnuplot 4.6.6 so I can test things. The original cset no longer applies cleanly to the source tree. Can you update it?

Rik <rik5>
Project Administrator
Sat 23 May 2015 12:27:03 PM UTC, comment #8:

Unmarking as postponed and retitling since gnuplot 5.0 has been out for a while now, and there may never be a 4.7.

Mike Miller <mtmiller>
Project Member
Sat 23 May 2015 06:36:06 AM UTC, comment #7:

Forgot about this one from long ago...

I don't think __gnuplot_has_feature__ is necessary in this case as < 4.7 appears to accept the limits being swapped and no "reverse/noreverse" specifier. I'm guessing the "reverse/noreverse" was removed from gnuplot because of the confusion that can result from syntax like this:

Should the "reverse" change the range to [0:4]?

If someone else wants to confirm that 4.6 works with the original change, that would save having to detect the feature. (Which is a slight problem, that I'm going to post to gnuplot discussion, but isn't important here.)

Dan Sebald <sebald>
Mon 28 Oct 2013 03:05:57 AM UTC, comment #6:

Marking as postponed since gnuplot 4.7 is not yet released.

Mike Miller <mtmiller>
Project Member
Thu 20 Jun 2013 06:18:26 PM UTC, comment #5:

Thanks. I'll follow up when 4.7 is released, but if I miss it feel free to apply your change set.

Dan Sebald <sebald>
Thu 20 Jun 2013 12:08:09 AM UTC, comment #4:

Dan, I modified your changeset slightly to use the function _gnuplot_has_feature_ which is our standard for querying whether gnuplot supports something. I'm attaching it to this bug report and you can use it if gnuplot does in fact go forward with their proposed modifications in 4.7.

(file #28360)

Rik <rik5>
Project Administrator
Wed 19 Jun 2013 10:56:39 PM UTC, comment #3:

Yes, it is particular to version 4.7 which has or hasn't come out yet. (I'm using the gnuplot development version on my system.) The behavior of "reverse" is going to change because it had a property of stickiness meaning once it was set, it wasn't possible to unset it. I eventually recalled having wondered about this on the gnuplot discussion list:

http://marc.info/?l=gnuplot-info-beta&m=134712826931471&w=2
http://marc.info/?t=124817327100003&r=1&w=2

I would have pursued this particular bug report (bug #38914) if I had felt comfortable with the change gnuplot developers are making, but the proposed change just didn't seem satisfactory so I'm just sort of taking a wait-and-see approach whether this is the final change when 4.7 comes out. 4.7 hasn't been released yet.

The attached patch does check for the version:

+ use_reverse_keyword = compare_versions (_gnuplot_version_ (), "4.7", "<");

so it is ready to go if gnuplot 4.7 is released and has the currently proposed change to "{no}reverse".

Dan Sebald <sebald>
Wed 19 Jun 2013 10:18:36 PM UTC, comment #2:

Is this particular to a specific version of gnuplot? I am using gnuplot 4.6.2 and the axis is correctly reversed for me with the existing code.

My test was

Rik <rik5>
Project Administrator
Tue 07 May 2013 12:45:30 AM UTC, comment #1:

As I was preparing the changeset I found a similar patch already on my computer. I must have fixed this a half year ago but didn't get around to submitting the changeset because there was a lot of other activity at the time with getting the GUI more in line with the signal/slot concept. I can't find any bug report that I created around the time of the original patch.

(file #28037)

Dan Sebald <sebald>
Tue 07 May 2013 12:20:05 AM UTC, original submission:

The "image" command is not producing an image with the y-axis reversed, although the matryoshka pointer info all looks correct.

The issue is that gnuplot graphics code is using the keyword "reverse"/"noreverse" but that only applies to automatic xrange specifications. When actual numerical values are given for the range (which is always the case here), the range values (e.g., xlim) need to be reversed.

I will attach a patch after submitting this report.

Dan Sebald <sebald>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #34085:  octave-gnuplot_axis_reverse-2015may23.patch added by sebald (2kB - application/octet-stream)
file #28360:  gnuplot_reverse_keyword.cset added by rik5 (4kB - application/octet-stream)
file #28037:  octave-gnuplot_range_change-2013may06.patch added by sebald (2kB - application/octet-stream)

 

Depends on the following items: None found

Digest:
   bug dependencies.

 

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

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 24 May 2015 04:55:14 AM UTCsebaldAttached File-=>Added octave-gnuplot_axis_reverse-2015may23.patch, #34085
    Sat 23 May 2015 12:27:03 PM UTCmtmillerStatusPostponed=>Patch Submitted
      SummaryGnuplot axis reverse is not working with gnuplot 4.7=>Gnuplot axis reverse is not working with gnuplot 5
    Sat 23 May 2015 12:23:34 PM UTCmtmillerDependencies-=>bugs #45167 is dependent
    Mon 28 Oct 2013 03:05:57 AM UTCmtmillerStatusWorks For Me=>Postponed
      SummaryGnuplot axis reverse is not working=>Gnuplot axis reverse is not working with gnuplot 4.7
    Thu 20 Jun 2013 12:08:09 AM UTCrik5Attached File-=>Added gnuplot_reverse_keyword.cset, #28360
    Wed 19 Jun 2013 10:18:36 PM UTCrik5StatusNone=>Works For Me
    Tue 07 May 2013 12:45:30 AM UTCsebaldAttached File-=>Added octave-gnuplot_range_change-2013may06.patch, #28037

    Back to the top


    Powered by Savane 3.1-cleanup