← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix for 3D model offset

 

What about a controversial idea:

Read "at" dimensions as inches, but new files write "offset" in mm.

This preserves read compatibility but fixes the units issue going forward.

On 9 Nov 2017 03:15, "jp charras" <jp.charras@xxxxxxxxxx> wrote:

> Le 08/11/2017 à 11:05, Oliver Walters a écrit :
> > 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
>
> Hi Oliver,
>
> Looks like currently, Pcbnew stores 3D offset in inches, unlike any other
> value.
> This is very annoying, but we have to leave with that.
>
> Therefore the patch breaks compatibility with any other Pcbnew version, if
> 3D offset is not 0.
>
>
> --
> Jean-Pierre CHARRAS
>
> _______________________________________________
> 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