← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix for 3D model offset

 

Wayne,

Another potential spanner in the works - the footprint files do not store
the file version information. (should they?)

Thus, footprint files cannot be parsed differently based on their file
version. Changing "at" between mm and inches will work for the pcb but not
for the individual footprint files.

On Sun, Nov 12, 2017 at 7:47 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx>
wrote:

> I misunderstood the intention of this patch.  I though it had nothing to
> do with the units change patch.  I will revert it.  Please send me the
> file format change patch when you get a chance and I merge them both at
> the same time.  Sorry about the mix up.
>
> On 11/11/2017 03:34 PM, Oliver Walters wrote:
> > Wayne,
> >
> > I had not yet made any changes to the file parser. Have you done this
> > according to your new file-version approach? Or do you want me to submit
> > a further patch implementing that?
> >
> > On 12 Nov 2017 07:16, "Wayne Stambaugh" <stambaughw@xxxxxxxxx
> > <mailto:stambaughw@xxxxxxxxx>> wrote:
> >
> >     Oliver,
> >
> >     I committed your patch to the KiCad development branch.
> >
> >     Thanks,
> >
> >     Wayne
> >
> >     On 11/08/2017 05:05 AM, Oliver Walters wrote:
> >     > Attached is a patch that fixes the problems I found in my 3D model
> >     array
> >     > investigation. As discussion on that is stalled for now, this patch
> >     > simply fixes the model offset issues.
> >     >
> >     > 1. Display offset units in 3D preview window
> >     >
> >     > - Offset units are displayed (either inches or mm)
> >     >
> >     > 2. Fix offset in 3D rendering
> >     >
> >     > - It appears that the internal units for 3D model offset (mm) were
> >     being
> >     > multiplied by 25.4 incorrectly
> >     > - Fixed rendering in OGL and Raytracing
> >     >
> >     > 3. Fix offset in 3D export
> >     >
> >     > - VRML export
> >     > - STEP export
> >     >
> >     > Oliver
> >     >
> >     >
> >     > _______________________________________________
> >     > Mailing list: https://launchpad.net/~kicad-developers
> >     <https://launchpad.net/~kicad-developers>
> >     > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >     > Unsubscribe : https://launchpad.net/~kicad-developers
> >     <https://launchpad.net/~kicad-developers>
> >     > More help   : https://help.launchpad.net/ListHelp
> >     <https://help.launchpad.net/ListHelp>
> >     >
> >
> >     _______________________________________________
> >     Mailing list: https://launchpad.net/~kicad-developers
> >     <https://launchpad.net/~kicad-developers>
> >     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> >     Unsubscribe : https://launchpad.net/~kicad-developers
> >     <https://launchpad.net/~kicad-developers>
> >     More help   : https://help.launchpad.net/ListHelp
> >     <https://help.launchpad.net/ListHelp>
> >
>

Follow ups

References