← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix for 3D model offset

 

Wayne,

Any further issues to address here? :)

On 14 Nov 2017 20:55, "Oliver Walters" <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>
> 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>:
>> > 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>> 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>>>
>> 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>>
>> >>     >     > 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>>
>> >>     >     >
>> >>     >
>> >>     >     _______________________________________________
>> >>     >     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>>
>> >>     >
>> >>
>> >>
>> >
>> > _______________________________________________
>> > 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