bugGNU Octave - Bugs: bug #39643, axes are not joined

 
 

bug #39643: axes are not joined

Submitted by:  Dmitri A. Sergatskov <dasergatskov>
Submitted on:  Thu Aug 1 12:49:02 2013  
 
Category: Plotting with OpenGLSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Other
Status: Patch SubmittedAssigned to: None
Originator Name: Dmitri A. SergatskovOpen/Closed: Open
Release: devOperating System: GNU/Linux

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)

Mon Nov 28 21:37:01 2016, comment #9:

I pushed the patch here:

http://hg.savannah.gnu.org/hgweb/octave/rev/1444a65085ca

The remaining issue is rendering line cap and/or line join properly onscreen. The approach proposed in bug #46739 (polygon based dashed lines) is probably the way to go.

Pantxo Diribarne <pantxo>
Project Member
Mon Nov 28 17:16:19 2016, comment #8:

Yes, test for gl2psLineJoin added in http://hg.savannah.gnu.org/hgweb/octave/rev/39f39eb4e476

Mike Miller <mtmiller>
Project Administrator
Mon Nov 28 16:44:33 2016, comment #7:

No objection to pushing the cset that is available now.

I notice you make use of HAVE_GL2PSLINEJOIN. Who sets that variable? Is there a test in configure.ac for it?

Rik <rik5>
Project Administrator
Mon Nov 28 11:45:35 2016, comment #6:

May I push the patch and leave this report open until we find a solution for onscreen rendering?

Pantxo Diribarne <pantxo>
Project Member
Tue Nov 22 15:41:28 2016, comment #5:

I attached the resulting figure from ML 2015b

My understanding is that having the line join/cap right onscreen is only possible if we change our way to draw lines: instead of using GL_LINE objects we should draw lines using polygons.

In any event I think the change I propose is the right way to go for now even though it will only be effective once a new version of gl2ps is released and available.

(file #39046)

Pantxo Diribarne <pantxo>
Project Member
Sun Nov 20 20:29:39 2016, comment #4:

I attached the png I obtain using Octave, the patch from comment #2 and the current dev version of gl2ps (see comment 27 in bug #48387).

(file #39033)

Pantxo Diribarne <pantxo>
Project Member
Sun Nov 20 18:20:17 2016, comment #3:

For reference, here is some sample test code:

Could someone try the above code on Matlab and upload the resulting image?

Rik <rik5>
Project Administrator
Fri Nov 18 16:08:08 2016, comment #2:

I was wondering if using LINE_STRIP to draw axes would do the trick for onscreen and printed figures, but LINE_STRIP which are used to draw line objects don't do a better job to join line segments (at least on my linux/nouveau driver):

I attached a patch that makes use of a new gl2psLineCap function in the current dev of gl2ps. This will thus only enhance the printout figure.

(file #39010)

Pantxo Diribarne <pantxo>
Project Member
Thu Aug 1 14:00:12 2013, comment #1:

Drawing is done through OpenGL. FLTK-based drawing is not involved.

Michael Goffioul <goffioul>
Project Member
Thu Aug 1 12:49:02 2013, original submission:

With fltk backend on Linux the axes lines are not joined in the corners. See attached for details; the axes width there set to 10 to make the problem more visible.

According to http://www.fltk.org/doc-1.3/drawing.html
<<<<
If you don't specify a cap or join type you will get a system-defined default of whatever value is fastest.

>>>>

so the bug might be OS (or even X11 server) dependent.

Dmitri A. Sergatskov <dasergatskov>

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #39046:  tst_linewidth.png added by pantxo (17kB - image/png)
file #39033:  tst_join.png added by pantxo (12kB - image/png)
file #39010:  linecap.patch added by pantxo (3kB - text/x-diff)
file #28733:  axes.png added by dasergatskov (9kB - image/png)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by pantxo (Updated the item)
  • -unavailable- added by goffioul (Posted a comment)
  • -unavailable- added by dasergatskov (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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue Nov 22 15:41:28 2016pantxoAttached File-=>Added tst_linewidth.png, #39046
    Sun Nov 20 20:29:39 2016pantxoAttached File-=>Added tst_join.png, #39033
    Fri Nov 18 16:08:08 2016pantxoAttached File-=>Added linecap.patch, #39010
      StatusNone=>Patch Submitted
    Thu Aug 1 12:49:02 2013dasergatskovAttached File-=>Added axes.png, #28733

    Back to the top


    Powered by Savane 3.1-cleanup