kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #39918
Re: [PATCH] Option to not render 3D models for footprints
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Wed, 27 Mar 2019 11:23:13 -0400
-
In-reply-to:
<CA+qGbCA=x2uE-aW16pb+--LshAnX34DAgGAhCtBr0VpY8Y3B2Q@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.6.0
I reviewed the original patch set and I would be willing to merge it but
I would prefer to use the existing file token "hide" rather than add a
new token "disabled". Also, the file format version will have to be
revised because JP's chamfered corners code has already been merged.
On 3/25/2019 4:55 PM, Jon Evans wrote:
> This is a good idea. I do this relatively frequently in Altium for
> cases like you described (battery holder + battery, shields, heatsinks, etc)
>
> On Mon, Mar 25, 2019 at 4:51 PM Rene Pöschl <poeschlr@xxxxxxxxx
> <mailto:poeschlr@xxxxxxxxx>> wrote:
>
> Maybe a better option would be to have footprint variations similar to
> aliases.
> Something with a new name, new 3d model path and new description but
> the
> same land pattern.
>
> This could be useful not only for having specialized 3d models but also
> when the manufacturer uses a very strange part naming scheme where it
> gets hard to define a footprint name well such that it tells the user
> which parts it exactly fits. (Example is if the part number code has
> for
> example a place that is either a,b.e for fp1 but c,d for fp2. right now
> we would need at least 3 possibly 5 footprints to properly do this.)
>
> The individual visibility stuff could also be useful for usecases that
> do not fit the one handled by my suggestion. One example would be a
> battery holder where you have the battery as a separate model. You
> might
> be interested in seeiing how it looks without a battery. Maybe even
> have
> a model where the battery is in the process of being inserted to see
> the
> space required for that task.
>
> Or a shield that covers some other parts. Might be useful to just hide
> this alone.
>
> On 25/03/19 19:46, Jeff Young wrote:
> > Multiple models is an existing feature (for building up parts).
> There’s just no individual control over visibility.
> >
> > Cheers,
> > Jeff.
> >
> >> On 25 Mar 2019, at 18:38, Wayne Stambaugh <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>> wrote:
> >>
> >> Sorry it took so long to get back to this but I've been really busy.
> >> The the capacitor example makes sense although I'm not sure this is a
> >> significant enough feature to warrant a file format change. I'm not
> >> terribly opposed to this idea either. I do have a few
> questions. Can
> >> multiple models be visible at the same time? If so, have the
> STEP and
> >> VRML exporters been tested to work under this case?
> >>
> >> Cheers,
> >>
> >> Wayne
> >>
> >> On 3/14/2019 10:26 AM, Jeff Young wrote:
> >>> Hi Wayne,
> >>>
> >>> No, it would need to be saved in the file. Think of it as Units
> for 3D
> >>> models: for instance you might have 30mm, 35mm and 40mm tall
> capacitors
> >>> all assigned to the single 20mm diameter 7.5mm pitch footprint.
> >>>
> >>> Cheers,
> >>> Jeff.
> >>>
> >>>
> >>>> On 14 Mar 2019, at 13:38, Wayne Stambaugh <stambaughw@xxxxxxxxx
> <mailto:stambaughw@xxxxxxxxx>
> >>>> <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>> wrote:
> >>>>
> >>>> Jeff,
> >>>>
> >>>> I haven't looked at Oliver's patch so I'm flying blind here. My
> >>>> question is why does this require a board change. Is this a
> state we
> >>>> really need to save in the board file or could it be some 3D viewer
> >>>> visibility state option saved in a config file? I would prefer the
> >>>> latter if possible. I guess I don't understand the purpose of
> this.
> >>>>
> >>>> Cheers,
> >>>>
> >>>> Wayne
> >>>>
> >>>> On 3/14/2019 6:44 AM, Jeff Young wrote:
> >>>>> @Wayne, this builds on top of my m_Preview addition so I’m
> happy to
> >>>>> review it and merge it after Oliver re-bases. But where do we
> stand on
> >>>>> PCBNew file format changes for 6.0? (There are also some
> hold-overs I
> >>>>> have from 5.1; namely storing defined diff pair dimensions and the
> >>>>> courtyard DRC settings in the files.
> >>>>>
> >>>>>> On 14 Mar 2019, at 08:30, Oliver Walters
> >>>>>> <oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>
> <mailto:oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>>
> >>>>>> <mailto:oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>>> wrote:
> >>>>>>
> >>>>>> This has gone unresolved for a while now - if I put in some
> effort to
> >>>>>> rebase this, is there any likelihood it will be accepted?
> >>>>>>
> >>>>>> This patchset does involve a file format change to the PCB
> file but it
> >>>>>> is backwards compatible and introduces a useful new feature.
> >>>>>>
> >>>>>> On Tue, Oct 30, 2018 at 11:27 PM Oliver Walters
> >>>>>> <oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>
> <mailto:oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>>
> >>>>>> <mailto:oliver.henry.walters@xxxxxxxxx
> <mailto:oliver.henry.walters@xxxxxxxxx>>> wrote:
> >>>>>>
> >>>>>> The attached patchset expands on the "Preview" checkbox
> in the 3D
> >>>>>> model tab in the footprint editor.
> >>>>>>
> >>>>>> This "Preview" option currently only applies to the preview
> >>>>>> window. However if the user wishes to disable display of
> a given
> >>>>>> 3D model in the PCB renderer they must delete the 3D
> model from
> >>>>>> the footprint entirely.
> >>>>>>
> >>>>>> The new patchset does the following:
> >>>>>>
> >>>>>> 1) The state of the m_Preview parameter for each 3D model is
> >>>>>> observed in the various 3D renderers and exporters
> >>>>>>
> >>>>>> 2) The m_Preview parameter is saved to file (both
> .kicad_mod and
> >>>>>> .kicad_pcb)
> >>>>>>
> >>>>>> With regard to file saving, if the 3D model is "enabled"
> (default
> >>>>>> state) then the file is unchanged making this change largely
> >>>>>> backwards compatible. If the 3D model is disabled, then the
> >>>>>> keyword "(disabled)" is added to the file.
> >>>>>>
> >>>>>> You can now quickly toggle 3D models on/off on an
> individual basis
> >>>>>> and this is statefully saved between sessions.
> >>>>>>
> >>>>>> Patch-set is rebased and compiled
> >>>>>> from b445b0fab28f7dd41273801d06d7705215c57c0f
> >>>>>>
> >>>>>> Regards,
> >>>>>>
> >>>>>> _______________________________________________
> >>>>>> Mailing list: 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>>
> >>>>>> Unsubscribe : https://launchpad.net/~kicad-developers
> >>>>>> More help : https://help.launchpad.net/ListHelp
> >>>>>
> >>>>> _______________________________________________
> >>>>> Mailing list: 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
> >>>>> More help : https://help.launchpad.net/ListHelp
> >>>>>
> >>>> _______________________________________________
> >>>> Mailing list: 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
> >>>> More help : https://help.launchpad.net/ListHelp
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help : https://help.launchpad.net/ListHelp
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : 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
>
References