← Back to team overview

kicad-developers team mailing list archive

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

 

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