bugGNU Octave - Bugs: bug #52412, Manual axes ticks should emit...

 
 

bug #52412: Manual axes ticks should emit error if values are not increasing

Submitted by:  Markus Mützel <mmuetzel>
Submitted on:  Wed 15 Nov 2017 10:11:10 AM UTC  
 
Category: PlottingSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Matlab Compatibility
Status: Patch SubmittedAssigned to: None
Originator Name: Open/Closed: Open
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 21 Nov 2017 12:49:39 PM UTC, comment #6:

The attached patch adds a new type of "special_constraints" to graphics properties (only row vector properties for now). I don't really like the name. But it pretty much describes what it can be used for: Adding constraints to properties that need "some special" check.
For now it adds the constraint "ENFORCE_ASCENDING" to the [xyz]tick and [xyz]lim properties of axes.
It can also be used to add other constraints in the future: e.g. the "position" (and related) properties should allow only non-negative values in the third and fourth element while there should be no such limitation on the first and second element...

I am not sure whether it makes sense to also integrate the (mutually exclusive) "finite_constraints" into these new (accumulative) "special_constraints".

I didn't find any properties that must be strictly decreasing. I just added that case to check whether the enumeration works as expected. If you think it is better to remove that case from the patch, I don't mind at all.

(file #42451)

Markus Mützel <mmuetzel>
Project Member
Wed 15 Nov 2017 11:22:51 PM UTC, comment #5:

Rik: Seems right to me.

Michael Godfrey <godfrey>
Project Member
Wed 15 Nov 2017 06:53:24 PM UTC, comment #4:

I also think it is reasonable to expect increasing values. I think that this catches a common case where a user mistypes or otherwise accidentally specifies a strange sequence of values.

If they really do have a strange sequence, it isn't a burden for them to sort those before passing to the plotting code.

Rik <rik5>
Project Administrator
Wed 15 Nov 2017 04:23:50 PM UTC, comment #3:

Just to be clear, I meant that the sort (Axes)
could go into the set (...) code for axis setting..

Michael Godfrey <godfrey>
Project Member
Wed 15 Nov 2017 10:32:17 AM UTC, comment #2:

Yes, I think it is reasonable to expect ticks to be increasing values.

Pantxo Diribarne <pantxo>
Project Member
Wed 15 Nov 2017 10:30:51 AM UTC, comment #1:

Seems pretty simple to just do a sort() on the
Axes vector. If there is a problem with this, then
Matlab "compatibility" should be the choice.

Michael Godfrey <godfrey>
Project Member
Wed 15 Nov 2017 10:11:10 AM UTC, original submission:

When setting manual axis ticks, Octave allows them in any order. Matlab throws the following error if they are not ascending:

While that could be a nice extension to the Matlab syntax, it might lead to unexpected behavior: e.g. the minor grid lines outside the major tick range are not shown (see bug #52376). There might be also other side effects.

Should we follow Matlab here and force the manual ticks to being in ascending order?

Markus Mützel <mmuetzel>
Project Member

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #42451:  bug52412_graphics_ascending_order.patch added by mmuetzel (9KiB - application/octet-stream)

 

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 pantxo (Posted a comment)
  • -unavailable- added by godfrey (Posted a comment)
  • -unavailable- added by mmuetzel (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
    Tue 21 Nov 2017 12:50:15 PM UTCmmuetzelStatusConfirmed=>Patch Submitted
    Tue 21 Nov 2017 12:49:39 PM UTCmmuetzelAttached File-=>Added bug52412_graphics_ascending_order.patch, #42451
    Wed 15 Nov 2017 06:53:24 PM UTCrik5StatusNone=>Confirmed
      SummaryManual axes ticks should be increasing=>Manual axes ticks should emit error if values are not increasing

    Back to the top


    Powered by Savane 3.1-cleanup1