Re: [Kicad-developers] Trapeziodal pads are buggy

2010-09-08 Thread jp.charras

Le 07/09/2010 17:58, Dick Hollenbeck a écrit :


On 09/07/2010 10:52 AM, Dick Hollenbeck wrote:

On 09/07/2010 09:17 AM, Dick Hollenbeck wrote:


I'm spending some time this morning looking into this.

Dick






Jean Pierre,

When adding the support for trapezoidal pads to specctra_export.cpp, I
noticed there is a bug in dialog_pad_properties.cpp which causes the
m_DeltaSize.y value to be displayed as positive even when the value has
been entered and saved previously as negative.

The same problem does not exist for the x value, which is displayed
properly as negative when it is.

In 4 minutes I could not find the source of the problem.

Dick



More info I forgot to mention:

this is the dialog which you call from the main pcbnew app, not from the
module editor.
The sign of the value is displayed properly when the dialog is invoked
from the module editor, but not properly when invoked from the main
program on a specific pad only.

Dick



In fact, this is not really a bug, but an inconsistency in pad editor, due to 
the way Pcbnew handles modules on back side.
When a footprint is flipped, the footprint is mirrored and some Y coordinates 
are negated.
So m_DeltaSize.y is negated by the mirroring transform and become negative if 
it was set to a positive value in module editor.
This is also true for m_Offset.y and m_Pos0.y
Sizes are always 0 and therefore are not modified.

The inconsistency is the fact the dialog uses values for m_DeltaSize.y and 
m_Offset.y
which are modified by the side where the footprint is,  but which are not 
modifed by its orientation.
(i.e. module orientation does not change the displayed values, but flipping 
this module change these values)

I am working on a dialog pad editor enhancement to avoid this inconsistency,
and also avoid mistakes in parameters edition.
(I think just modifying the way Delta Y size or Offset Y value is displayed is 
not enough)

Note also trapezoidal pads are mainly for microwaves applications and others issues (clearance area shape, DRC...) are more a not 
finished work problem than a bug.

DRC using microwaves constraints is a lot more tricky than usual DRC

--
Jean-Pierre CHARRAS
KiCad Developers team.
KiCad Developers kicad-developers@lists.launchpad.net

___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp


Re: [Kicad-developers] Trapeziodal pads are buggy

2010-09-07 Thread Dick Hollenbeck
On 09/07/2010 10:52 AM, Dick Hollenbeck wrote:
 On 09/07/2010 09:17 AM, Dick Hollenbeck wrote:
   
 I'm spending some time this morning looking into this.

 Dick

   
 


 Jean Pierre,

 When adding the support for trapezoidal pads to specctra_export.cpp, I
 noticed there is a bug in dialog_pad_properties.cpp which causes the
 m_DeltaSize.y value to be displayed as positive even when the value has
 been entered and saved previously as negative.

 The same problem does not exist for the x value, which is displayed
 properly as negative when it is.

 In 4 minutes I could not find the source of the problem.

 Dick
   

More info I forgot to mention:

this is the dialog which you call from the main pcbnew app, not from the
module editor.
The sign of the value is displayed properly when the dialog is invoked
from the module editor, but not properly when invoked from the main
program on a specific pad only.

Dick


___
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp