← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Option to not render 3D models for footprints

 

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> 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> 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>> 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>> 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>> 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>
> >>>>>> 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
>
>
>
> _______________________________________________
> 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