← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] pcbnew export position file

 

Hey Brian,

Go ahead and commit this.  I'm just about ready to pull the trigger on
the changes required to build the full MSYS2 package.

Thanks,

Wayne

On 10/31/2014 10:25 AM, Brian Sidebotham wrote:
> Hi Wayne,
> 
> It was on tonights list (for me that's around 6~7hrs from now). If
> you've got time, by all means put it through now, that would be great.
> 
> Best Regards,
> 
> Brian.
> 
> On 31 October 2014 13:59, Wayne Stambaugh <stambaughw@xxxxxxxxxxx> wrote:
>> On 10/29/2014 8:47 AM, Brian Sidebotham wrote:
>>> On 29 October 2014 09:15, Michal Jahelka <michal.jahelka@xxxxxxxxxx> wrote:
>>>> Hello,
>>>>
>>>> I found small problem when generating position file. It was good before
>>>> complete library name was used. It is shorted, and it is mainly impossible
>>>> to understand which package has any footprint - there is mainly only part of
>>>> library name. for example:
>>>>
>>>> # Ref    Val                  Package         PosX       PosY        Rot
>>>> Side
>>>> C5       1u               Capacitor-sm_cap   -0.1800    -1.2100     270.0
>>>> F.Cu
>>>> C8       100u/6.3VB       Capacitor-sm_cap   -0.1800    -1.0300     180.0
>>>> F.Cu
>>>> C9       100n             Capacitor-sm_cap   -0.1000     0.2150     180.0
>>>> F.Cu
>>>> C12      100n             Capacitor-sm_cap   -0.0250     0.9550       0.0
>>>> F.Cu
>>>> C13      100n             Capacitor-sm_cap    0.3200     0.8050      90.0
>>>> F.Cu
>>>> C14      100n             Capacitor-sm_cap    0.4700     0.5100     270.0
>>>> F.Cu
>>>> C16      100n             Capacitor-sm_cap   -0.4600     1.2600       0.0
>>>> F.Cu
>>>> C17      T22u/10V         Capacitor-sm_cap    0.1000     0.2400       0.0
>>>> F.Cu
>>>> C18      1u               Capacitor-sm_cap   -0.1250     1.0600     180.0
>>>> F.Cu
>>>> C19      100n             Capacitor-sm_cap   -0.4500    -0.8150     180.0
>>>> F.Cu
>>>> Q1       IRLML0030        libcms:SOT23GDS     0.0950    -0.6950     180.0
>>>> F.Cu
>>>> U1       LP2951           Vlastni-SM-IC:SO   -0.4000    -1.2100      90.0
>>>> F.Cu
>>>>
>>>> So after patch is visible only package, without library (I think that for
>>>> machine component placing is library name not important).
>>>>
>>>> # Ref    Val                  Package         PosX       PosY        Rot
>>>> Side
>>>> C5       1u               c_1206             -0.1800    -1.2100     270.0
>>>> F.Cu
>>>> C8       100u/6.3VB       c_tant_B           -0.1800    -1.0300     180.0
>>>> F.Cu
>>>> C9       100n             c_0805             -0.1000     0.2150     180.0
>>>> F.Cu
>>>> C12      100n             c_0805             -0.0250     0.9550       0.0
>>>> F.Cu
>>>> C13      100n             c_0805              0.3200     0.8050      90.0
>>>> F.Cu
>>>> C14      100n             c_0805              0.4700     0.5100     270.0
>>>> F.Cu
>>>> C16      100n             c_0805             -0.4600     1.2600       0.0
>>>> F.Cu
>>>> C17      T22u/10V         c_tant_B            0.1000     0.2400       0.0
>>>> F.Cu
>>>> C18      1u               c_1206             -0.1250     1.0600     180.0
>>>> F.Cu
>>>> C19      100n             c_0805             -0.4500    -0.8150     180.0
>>>> F.Cu
>>>> Q1       IRLML0030        SOT23GDS            0.0950    -0.6950     180.0
>>>> F.Cu
>>>> U1       LP2951           SOG8               -0.4000    -1.2100      90.0
>>>> F.Cu
>>>>
>>>> Michal Jahelka
>>>>
>>>
>>> I agree. The package is what matters, the library that comes from is
>>> unknown to most people anyway. It is only a hint as to what to expect.
>>> The start of the library name doesn't really give you any hint at all.
>>>
>>> Perhaps some people aggregate all of the different "Packages" and
>>> "Val" combinations there are to roughly determine the number of reels
>>> they'll need to use on the machine.
>>>
>>> If no-one objects I'll commit the change for you.
>>>
>>> Best Regards,
>>>
>>> Brian.
>>>
>>
>> Brian,
>>
>> Are you going to commit this patch soon?  I think it makes sense not to
>> include the library nickname in the position file.  If not, I have some
>> time to take care of it.
>>
>> Thanks,
>>
>> Wayne
>>
>>
>> _______________________________________________
>> 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
> 



Follow ups

References