← Back to team overview

kicad-developers team mailing list archive

Re: Need feedback on new way of pcbnew importing footprints

 

Bernhard,

I tried to replicate the bug and I think it happen different than you are
thinking.

What I did was I opened a board I already had, swapped a net on a footprint
that was already on the board, generated the netlist and imported it with
those settings, and it wasn't in move mode.

What I'm guessing is when you clicked "yes" to the pop up after reading the
netlist that the footprint was under the cursor at that time. Apparently if
you invoke the move selection tool and there are no selected items it will
try to pick an item under the cursor.

A patch to fix that behavior was committed just a bit ago. You are welcome
to test that, or if you can repeat the bug you are mentioning I have a test
for you to do:
Swap a net on the footprint, but also add a new component/footprint at the
same time. Then when you import the netlist only the new footprint should
be in move mode.

Jon

On Sun, Jan 24, 2016 at 4:16 PM Bernhard Stegmaier <stegmaier@xxxxxxxxxxxxx>
wrote:

> Great, thanks!
> This is really a very useful feature.
>
>
> Regards,
> Bernhard
>
>
> On 24 Jan 2016, at 22:14, Jon Neal <reportingsjr@xxxxxxxxx> wrote:
>
> Hmmm, I think I know where to look for this. I'll check it out later
> tonight.
>
> Thanks for the heads up Bernhard!
>
> Jon
>
> On Sun, Jan 24, 2016 at 4:09 PM Bernhard Stegmaier <
> stegmaier@xxxxxxxxxxxxx> wrote:
>
>> One minor thing I just found, don’t know if it is intended this way:
>>
>> I have some IC on my board, most of its connections already routed.
>> I swapped one pin of it to a different net in eeschema, exported the
>> netlist.
>> In pcbnew I imported the netlist (Footprint Selection = Reference,
>> Exchange Footprint = Change, everything else = Delete).
>> After closing the netlist dialog the IC was in move mode just like if it
>> was a new component.
>>
>> I wouldn’t have expected this, since it is not a new component.
>>
>>
>> Regards,
>> Bernhard
>>
>>
>> > On 10 Jan 2016, at 22:28, Chris Pavlina <pavlina.chris@xxxxxxxxx>
>> wrote:
>> >
>> > Patch committed in 6439. I like this a lot, it doesn't seem to break
>> > anything*, and it's fairly minimal in reach. Thank you!
>> >
>> > *I also can't see any possible way it could have caused the reported
>> > library issue, that's surely unrelated.
>> >
>> > --
>> > Chris
>> >
>> > On Tue, Jan 05, 2016 at 09:10:04PM +0000, Jon Neal wrote:
>> >> here is the patch:
>> >>
>> https://github.com/reportingsjr/kicad-source-mirror/commit/dd96d250321b17e0f8691d5623ea202f8762023c.patch
>> >>
>> >> On Tue, Jan 5, 2016 at 3:03 PM Jon Neal <reportingsjr@xxxxxxxxx>
>> wrote:
>> >>
>> >>> If no one has any complaints or suggestions then this patch is ready
>> to be
>> >>> included.
>> >>>
>> >>> I can generate a git diff with commit, author etc if needed, just let
>> me
>> >>> know.
>> >>>
>> >>> Also, I accidentally sent this to Maciej only earlier:
>> >>> Simon Richter mentioned that doing a swap was a cleaner method of
>> doing it
>> >>> for a variety of reasons. It doesn't really have to be done that way,
>> just
>> >>> following his advice.
>> >>>
>> >>> Jon
>> >>>
>> >>> On Tue, Jan 5, 2016 at 10:55 AM Andy Peters <devel@xxxxxxxxx> wrote:
>> >>>
>> >>>>
>> >>>>> On Jan 4, 2016, at 6:59 PM, Jon Neal <reportingsjr@xxxxxxxxx>
>> wrote:
>> >>>>>
>> >>>>> I asked about this a while back and got a little bit of feedback, I
>> now
>> >>>> have a patch for people to try.
>> >>>>>
>> >>>>> What this does is when you import a netlist (in GAL only though) it
>> >>>> automatically spreads out all newly added footprints and it places
>> them all
>> >>>> in "selection + move" mode. So basically you import, close the import
>> >>>> window, and immediately click wherever to plonk down the new
>> footprints.
>> >>>>>
>> >>>>> I want to know if people like this method of importing (I think it
>> is
>> >>>> nicer than piling all of the footprints on top of each other
>> wherever).
>> >>>>
>> >>>>
>> >>>> YES!
>> >>>>
>> >>>> -a
>> >>>>
>> >>>>
>> >>>>
>> >>>> _______________________________________________
>> >>>> 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