← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] pcbnew export position file

 

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



Follow ups

References