Thick lines are longer when printed

Bug #698603 reported by phil_frost
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gEDA
Fix Released
Low
Unassigned

Bug Description

I've noticed when I export schematics with thick lines to postscript, those thick lines are longer than they would be if they were thin. See the attached screenshots for example. The line in that symbol representing the gate is thickness = 40, while the three shorter lines for the source, drain, and body are thickness = 20. Viewed in gschem, the top and bottom of the gate line align horizontally with the other lines. Viewed in postscript, the gate line is longer.

It's especially problematic with the MOSFET symbol I'm trying to make here. I'd like to make the drain, source, and body lines thicker. If I do that, they get long enough to merge together. If I make them shorter so they look right in postscript, the symbol looks pretty goofy in the GUI.

Revision history for this message
phil_frost (phil-frost) wrote :
Revision history for this message
phil_frost (phil-frost) wrote :
Revision history for this message
phil_frost (phil-frost) wrote :
Revision history for this message
phil_frost (phil-frost) wrote :

Ah yes, and version is 1.6.1.20100214 on Mac OS, built from source. Have the same problem on 1.6.0 on Ubuntu.

Revision history for this message
Peter Clifton (pcjc2) wrote :

Has this behaviour changed between 1.4.x and 1.6.x ?

I'm vaguely of the mind that it is the GUI which needs fixing, since I don't believe the postscript code has changed - I presume Phil, that you're running the official source version with no patches - which still uses the old postscript printing code.

Revision history for this message
Krzysztof Kościuszkiewicz (k-kosciuszkiewicz) wrote :

I can still reproduce this with 1.7.0.
This occurs when line end types do not match with the gschemrc setting for "output-capstyle" setting.
Current postscript output code overrides line end settings from gschem with the type specified with the config option.

Changed in geda:
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Peter Clifton (pcjc2) wrote :

This will be resolved when gschem moves to using the same cairo rendering code for printing as it does for rendering. Assigning Peter Brett, as he is working on that code.

tags: added: printing
Changed in geda:
assignee: nobody → Peter TB Brett (peter-b)
milestone: none → 1.8.0
Revision history for this message
Peter TB Brett (peter-b) wrote :

BTW, that's under development in my 'libgedacairo' branch <http://repo.or.cz/w/geda-gaf/peter-b.git>. Take a look, if you want. Warning: not currently feature-complete, likely to asplode, use with caution.

Revision history for this message
Peter TB Brett (peter-b) wrote :

This isn't going to make it in time for 1.8.0, so I've removed it from that milestone.

Changed in geda:
milestone: 1.8.0 → none
Peter TB Brett (peter-b)
Changed in geda:
status: Confirmed → Triaged
Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :

I've made a patchset for current master git head, which solves this bug. See attachments. Last of them is a schematic to test objects.

Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :
Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :
Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :
Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :
Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :

Test schematic ;)

Revision history for this message
Peter TB Brett (peter-b) wrote :

Do you have a branch I can pull from?

Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :

No public branches, only local.
And I have some more related fixes (a couple). They are not for this bug, however.

Revision history for this message
Vladimir Zhbanov (vzhbanov) wrote :

First two patches are also related to bug #789427.

Revision history for this message
Peter TB Brett (peter-b) wrote :

I've committed this patch to master.

Changed in geda:
assignee: Peter TB Brett (peter-b) → nobody
status: Triaged → Fix Committed
Peter TB Brett (peter-b)
Changed in geda:
milestone: none → 1.9.0
Peter TB Brett (peter-b)
Changed in geda:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.