kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #31773
Re: [PATCH] Fix for 3D model offset
Oliver,
I merged these patches into the development branch.
Thanks,
Wayne
On 11/20/2017 04:35 PM, Oliver Walters wrote:
> Wayne,
>
> Patch files attached.
>
> Once again the --ignore-whitespace flag will need to be used, I believe
> it is because a line in kicad-plugin.h (46, corresponding to 20170920)
> has a DOS newline instead of a UNIX newline.
>
>
> Thanks,
>
> On Tue, Nov 21, 2017 at 4:27 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>> wrote:
>
> Oliver,
>
> Please send me the latest patch(es) when you get a chance so I can get
> this merged. I cannot find it (them) in my inbox. I must have deleted
> the email by accident.
>
> Cheers,
>
> Wayne
>
> On 11/18/2017 8:39 PM, Oliver Walters wrote:
> > Wayne,
> >
> > Any further issues to address here? :)
> >
> > On 14 Nov 2017 20:55, "Oliver Walters" <oliver.henry.walters@xxxxxxxxx <mailto:oliver.henry.walters@xxxxxxxxx>
> > <mailto:oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>>> wrote:
> >
> > Wayne,
> >
> > Please find attached updated patch set. If an old version is
> > detected, inches are converted to mm.
> >
> > On Tue, Nov 14, 2017 at 9:33 AM, Nick Østergaard <oe.nick@xxxxxxxxx <mailto:oe.nick@xxxxxxxxx>
> > <mailto:oe.nick@xxxxxxxxx <mailto:oe.nick@xxxxxxxxx>>> wrote:
> >
> > Just to mention it, I think this quirk needs to be put in the
> > release
> > notes for 5.0.0 if that would be the case, or at least
> > documented as a
> > note in the docs.
> >
> > 2017-11-13 18:10 GMT+01.00, Wayne Stambaugh
> > <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>
> <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>>:
> > > On 11/12/2017 6:12 AM, Oliver Walters wrote:
> > >> Wayne,
> > >>
> > >> Another potential spanner in the works - the footprint
> files
> > do not
> > >> store the file version information. (should they?)
> > >
> > > I would rather not. Otherwise there would be a footprint
> > version in
> > > every footprint embedded in the schematic and could lead to
> > some ugly
> > > parser code. At some point we may need to revisit this but
> > not now.
> > >
> > > This is not a big issue because the only effects the
> > footprints embedded
> > > in the board. Users with custom footprint libraries that
> > contain 3D
> > > model offsets will just have to fix the offsets. I'm
> guessing
> > this is a
> > > fairly small number of users.
> > >
> > >>
> > >> 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 <mailto:stambaughw@xxxxxxxxx>
> <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>
> > >> <mailto:stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx> <mailto:stambaughw@xxxxxxxxx
> <mailto: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>
> <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>
> > >> <mailto:stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx> <mailto:stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>>>
> > >> > <mailto:stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>
> > <mailto:stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>> <mailto:stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>
> > <mailto: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>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>
> > >> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>>
> > >> > > Post to :
> > kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> > >> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>>
> > >> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> > >> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>>>
> > >> > > Unsubscribe :
> > https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>
> > >> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>>
> > >> > > More help :
> https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>
> > >> <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>>
> > >> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>
> > >> <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>>>
> > >> > >
> > >> >
> > >> > _______________________________________________
> > >> > Mailing list:
> > https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>
> > >> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>>
> > >> > Post to :
> kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> > >> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>>
> > >> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> > >> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>>>
> > >> > Unsubscribe :
> > https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>
> > >> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > >> <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>>>
> > >> > More help :
> https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>
> > >> <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>>
> > >> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>
> > >> <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>>>
> > >> >
> > >>
> > >>
> > >
> > > _______________________________________________
> > > Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>>
> > > Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> > <https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>>
> > > More help : https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>
> > <https://help.launchpad.net/ListHelp
> <https://help.launchpad.net/ListHelp>>
> > >
> >
> >
>
>
References
-
[PATCH] Fix for 3D model offset
From: Oliver Walters, 2017-11-08
-
Re: [PATCH] Fix for 3D model offset
From: Wayne Stambaugh, 2017-11-11
-
Re: [PATCH] Fix for 3D model offset
From: Oliver Walters, 2017-11-11
-
Re: [PATCH] Fix for 3D model offset
From: Wayne Stambaugh, 2017-11-11
-
Re: [PATCH] Fix for 3D model offset
From: Oliver Walters, 2017-11-12
-
Re: [PATCH] Fix for 3D model offset
From: Wayne Stambaugh, 2017-11-13
-
Re: [PATCH] Fix for 3D model offset
From: Nick Østergaard, 2017-11-13
-
Re: [PATCH] Fix for 3D model offset
From: Oliver Walters, 2017-11-14
-
Re: [PATCH] Fix for 3D model offset
From: Oliver Walters, 2017-11-19
-
Re: [PATCH] Fix for 3D model offset
From: Wayne Stambaugh, 2017-11-20
-
Re: [PATCH] Fix for 3D model offset
From: Oliver Walters, 2017-11-20