haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

PNS: correct initial segment direction for posture solver

This commit is contained in:
Jon Evans 2020-12-31 13:03:52 -05:00
parent 111c644a82
commit 73248ab219
2 changed files with 15 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
libs/kimath/include/geometry/seg.h
View File

@ -354,6 +354,11 @@ public:
std::swap( A, B );
}
SEG Reversed() const
{
return SEG( B, A );
}
///> Returns the center point of the line
VECTOR2I Center() const
{

12
pcbnew/router/pns_line_placer.cpp
View File

@ -987,14 +987,22 @@ bool LINE_PLACER::Start( const VECTOR2I& aP, ITEM* aStartItem )
if( aStartItem && aStartItem->Kind() == ITEM::SEGMENT_T )
{
// If we land on a segment endpoint, assume the starting direction is continuing along
// the same direction as the endpoint. If we started in the middle, don't set a
// direction so that the posture solver is not biased.
SEG seg = static_cast<SEGMENT*>( aStartItem )->Seg();
// NOTE: have to flip this over at the moment because DIRECTION_45(SEG) assumes +y is
// North but a SEG will have -y be North in KiCad world coordinate system.
// This should probably be fixed in DIRECTION_45 but it's used in a lot of PNS code
// that would need to be checked carefully for such a change...
SEG seg = static_cast<SEGMENT*>( aStartItem )->Seg();
seg.A.y = -seg.A.y;
seg.B.y = -seg.B.y;
lastSegDir = DIRECTION_45( seg );
if( aP == seg.A )
lastSegDir = DIRECTION_45( seg );
else if( aP == seg.B )
lastSegDir = DIRECTION_45( seg.Reversed() );
}
else if( aStartItem && aStartItem->Kind() == ITEM::SOLID_T &&
static_cast<SOLID*>( aStartItem )->Parent()->Type() == PCB_PAD_T )