← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Add bidirectional net highlight cross-probing

 

I'm putting my project leader hat.  Since the bug was not a wishlist bug
I will go ahead and merge the original patch along with the board
centering fix.  We will push the schematic view centering which is a far
more complex issue of until v6.

@John, I merged your first patch.  Thank you.  I will try to get the
board centering patch tested and merged today.

Cheers,

Wayne

On 01/07/2018 09:56 AM, Chris Pavlina wrote:
> I agree, don't do that. And yes, it's really complicated because nets
> can span sheets. Just highlighting it in the existing viewport is
> perfectly fine and what I'd expect.
> 
> On Sun, Jan 07, 2018 at 02:50:27PM +0000, Jon Evans wrote:
>> Now we are really straying into new-feature land...  as far as I can tell,
>> there is no code in eeschema for "locate and zoom to fit a net".
>> I could pick some random object on the net and locate that without much
>> code, but is that the behavior we want? (especially for nets that exist on
>> multiple sheets)
>>
>> -Jon
>>
>> On Sun, Jan 7, 2018 at 9:02 AM, Nick Østergaard <oe.nick@xxxxxxxxx> wrote:
>>
>>> Hi Jon,
>>>
>>> I just tested this woth both patches applied. It seems that selecting nets
>>> in eeschema the zoom to view works good in pcbnew, but the other way around
>>> not so well. It does not always zoom to the net in eeschema when selecing
>>> nets in pcbnew.
>>>
>>> 2018-01-07 3:30 GMT+01:00 Jon Evans <jon@xxxxxxxxxxxxx>:
>>>
>>>> Hi Wayne,
>>>>
>>>> Give this patch a try on top of my first one and see if it does what you
>>>> want.
>>>>
>>>> -Jon
>>>>
>>>> On Sat, Jan 6, 2018 at 8:58 PM, Chris Pavlina <pavlina.chris@xxxxxxxxx>
>>>> wrote:
>>>>
>>>>> On Sun, Jan 07, 2018 at 01:48:34AM +0000, Jon Evans wrote:
>>>>>> Do I get a pass if I did this because I was browsing the bug list
>>>>> looking
>>>>>> for more things to fix? ;-)
>>>>>
>>>>> I suppose so, I disagree with it having been classified as a bug but
>>>>> that's not your fault ;)
>>>>>
>>>>>>
>>>>>> Wayne, I'll work on the recentering in PcbNew, it shouldn't be too
>>>>> much to
>>>>>> add.
>>>>>>
>>>>>> On Sat, Jan 6, 2018 at 7:45 PM, Chris Pavlina <pavlina.chris@xxxxxxxxx
>>>>>>
>>>>>> wrote:
>>>>>>
>>>>>>> I like it too, I just worry that by accepting even small features
>>>>> during
>>>>>>> freeze we're encouraging people to do things that aren't bugfixes and
>>>>>>> release cleanup...
>>>>>>>
>>>>>>> On Sun, Jan 07, 2018 at 12:35:58AM +0000, Wayne Stambaugh wrote:
>>>>>>>> Jon,
>>>>>>>>
>>>>>>>> I just tested this and I like it!  It doesn't seem like a high risk
>>>>>>>> feature so if now one objects I'm going to merge it.  If you have
>>>>> time,
>>>>>>>> I do have one minor request.  When highlighting a net in the board
>>>>>>>> editor, if the net in the schematic editor is scrolled of the
>>>>> display,
>>>>>>>> the schematic editor centers the net so you can see it.  The same
>>>>> thing
>>>>>>>> does not happen when a net is highlighted in the schematic editor
>>>>> and
>>>>>>>> the net on the board is scrolled off the display.  Would you
>>>>> please fix
>>>>>>>> this so the net in the board editor is centered on the display
>>>>> when the
>>>>>>>> net in the schematic editor is selected.
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>>
>>>>>>>> Wayne
>>>>>>>>
>>>>>>>> On 01/06/2018 03:52 PM, Jon Evans wrote:
>>>>>>>>> I implemented this in response to a bug report:
>>>>>>>>> https://bugs.launchpad.net/kicad/+bug/1738875
>>>>>>>>>
>>>>>>>>> But it is kind of on a blurred line between bug fix and new
>>>>> feature.
>>>>>>>>> I'll leave it up to Wayne et. al. to decide on if to merge this
>>>>> for V5.
>>>>>>>>>
>>>>>>>>> Basically, beforehand, cross-probing highlighted nets only if you
>>>>>>>>> clicked on a component pad.
>>>>>>>>> Now there is a separate cross-probing message for a net name, so
>>>>> that
>>>>>>> if
>>>>>>>>> the highlight tool is selected in both eeschema and pcbnew,
>>>>> selecting
>>>>>>>>> any part of a net will cross-probe.
>>>>>>>>>
>>>>>>>>> -Jon
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> _______________________________________________
>>>>>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>>>>>
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>>>>> More help   : https://help.launchpad.net/ListHelp
>>>>>>>
>>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>> More help   : https://help.launchpad.net/ListHelp
>>>>
>>>>
>>>
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 


References