← Back to team overview

kicad-developers team mailing list archive

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