← Back to team overview

kicad-developers team mailing list archive

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

 

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