Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-05 Thread Mark Roszko
As a FYI, Coverity has generated defects related to the new features that seem like they can generate unexpected behavior from what I see: #106428 (variable not initialized, considered high impact) #106367 (null dereference) #106366 (logically dead code) there's a bunch of other defects but mostl

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-05 Thread Simon Richter
Hi, On 06.03.2015 02:53, Mark Roszko wrote: > /home/mroszko/kicad/include/math/vector2d.h:412:54: warning: > ‘*((void*)(& m)+4).VECTOR2::y’ may be used uninitialized in this > function [-Wmaybe-uninitialized] > return VECTOR2 ( x - aVector.x, y - aVector.y ); That is an interesting warning

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-05 Thread Mark Roszko
FYI, while compiling 5481 for a coverity scan In file included from /home/mroszko/kicad/include/geometry/shape.h:28:0, from /home/mroszko/kicad/pcbnew/router/pns_diff_pair.cpp:27: /home/mroszko/kicad/include/math/vector2d.h: In member function ‘void PNS_DP_GATEWAYS::BuildGeneric(

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-05 Thread Tomasz Wlostowski
On 05.03.2015 10:20, LordBlick wrote: > In response to a message written on 03.03.2015, 20:22, from Tomasz > Wlostowski: >> The pads of the top/bottom connector on your PCB don't have the same X >> coordinates (for the top connector X = 148,5011 mm, for the bottom >> connector X = 148,5 mm). > > I

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-05 Thread Brian Sidebotham
On 3 March 2015 at 13:44, Tomasz Wlostowski wrote: > Dear all, > > We've just merged the new features of the Interactive Router to the > product branch: > - Routing differential pairs (in push&shove mode too). > - Tuning trace length. > - Tuning differential pair length and skew. > > There's a vid

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-05 Thread LordBlick
In response to a message written on 03.03.2015, 20:22, from Tomasz Wlostowski: The pads of the top/bottom connector on your PCB don't have the same X coordinates (for the top connector X = 148,5011 mm, for the bottom connector X = 148,5 mm). In my opinion (if I understand idea) in some particul

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Miguel Ángel Ajo
A-ma-zing work! ;) Thank you. Miguel Ángel Ajo On Wednesday, 4 de March de 2015 at 3:38, Andrew Zonenberg wrote: > So if I understand this correctly, net names must currently be FOO+ and > FOO- for differential pair routing. > > Can you add support for FOO_P and FOO_N naming? These names

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Nick Østergaard
2015-03-04 5:20 GMT+01:00 Jon Neal : > In the video Tom mentions that _N and _P work as well. > > This is an awesome addition! Now if only this had existed last summer I > could have saved a lot of time. :) Now you can redo your board, and get a feel for how much time you cold have saved... :) >

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Jon Neal
In the video Tom mentions that _N and _P work as well. This is an awesome addition! Now if only this had existed last summer I could have saved a lot of time. :) On Tue, Mar 3, 2015 at 10:10 PM, Blair Bonnett wrote: > On 4 March 2015 at 15:38, Andrew Zonenberg > wrote: > > > > So if I understa

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Blair Bonnett
On 4 March 2015 at 15:38, Andrew Zonenberg wrote: > > So if I understand this correctly, net names must currently be FOO+ and FOO- for differential pair routing. > > Can you add support for FOO_P and FOO_N naming? _P and _N works for me in r5472. ___ Ma

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Andrew Zonenberg
So if I understand this correctly, net names must currently be FOO+ and FOO- for differential pair routing. Can you add support for FOO_P and FOO_N naming? These names are legal identifiers in most hardware description languages and will allow PCB and HDL net names to be the same. On 03/03/2

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Nick Østergaard
2015-03-03 20:22 GMT+01:00 Tomasz Wlostowski : > On 03.03.2015 18:53, Nick Østergaard wrote: >> 2015-03-03 18:46 GMT+01:00 Nick Østergaard : >>> 2015-03-03 18:28 GMT+01:00 Tomasz Wlostowski : On 03.03.2015 18:12, Nick Østergaard wrote: > I think I found a bug. > > I made a test pro

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Tomasz Wlostowski
On 03.03.2015 18:53, Nick Østergaard wrote: > 2015-03-03 18:46 GMT+01:00 Nick Østergaard : >> 2015-03-03 18:28 GMT+01:00 Tomasz Wlostowski : >>> On 03.03.2015 18:12, Nick Østergaard wrote: I think I found a bug. I made a test project with two connectors, named the nets like a+ and >>

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Nick Østergaard
2015-03-03 18:46 GMT+01:00 Nick Østergaard : > 2015-03-03 18:28 GMT+01:00 Tomasz Wlostowski : >> On 03.03.2015 18:12, Nick Østergaard wrote: >>> I think I found a bug. >>> >>> I made a test project with two connectors, named the nets like a+ and >>> a-. Then I use the Route menu -> Differential Pa

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Nick Østergaard
I think I found a bug. I made a test project with two connectors, named the nets like a+ and a-. Then I use the Route menu -> Differential Pair, start at one of my pairs, that is directly south of the original pads... I expected it to connect, but it does not. I needed to make a "kink" or cornor

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Nick Østergaard
2015-03-03 18:28 GMT+01:00 Tomasz Wlostowski : > On 03.03.2015 18:12, Nick Østergaard wrote: >> I think I found a bug. >> >> I made a test project with two connectors, named the nets like a+ and >> a-. Then I use the Route menu -> Differential Pair, start at one of >> my pairs, that is directly so

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Tomasz Wlostowski
On 03.03.2015 18:12, Nick Østergaard wrote: > I think I found a bug. > > I made a test project with two connectors, named the nets like a+ and > a-. Then I use the Route menu -> Differential Pair, start at one of > my pairs, that is directly south of the original pads... I expected it > to connec

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Nick Østergaard
Looks very nice, I look forward to trying it out when I get home. I will use the possibility to mention that, ... if you fell like writing docs, don't forget what you wrote in [2] and merge it with the effort in [3] when updated. [2] http://bazaar.launchpad.net/~kicad-product-committers/kicad/p

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Wayne Stambaugh
Great work Tom and Orson and many thanks to CERN for making this possible. Please test the new diff pair router as much as possible so we can get any issues sorted out as we push towards a stable release. I know this merge fixed some outstanding issues in the OpenGL mode so please tag any bugs th

Re: [Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Javier Serrano
On Tue, Mar 3, 2015 at 2:44 PM, Tomasz Wlostowski wrote: > Many thanks to all developers, funders & users for your help and support! Thanks in particular to you, Tom, great job! And to the Raspberry Pi Foundation for an important donation which has greatly helped in this development. Cheers, Ja

[Kicad-developers] Diff Pairs & Length tuning in the product branch

2015-03-03 Thread Tomasz Wlostowski
Dear all, We've just merged the new features of the Interactive Router to the product branch: - Routing differential pairs (in push&shove mode too). - Tuning trace length. - Tuning differential pair length and skew. There's a video tutorial and a demo available on YouTube [1]. The merge also inc