← Back to team overview

kicad-developers team mailing list archive

Re: OCE plugin for 3D viewer

 

> On 04 Sep 2016, at 22:58, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
> 
> On 9/4/2016 3:52 PM, Chris Pavlina wrote:
>> PLEASE do not rename these options to just "ifdef OSX". The current
>> names are explantory, they make it clear WHY sections of code are
>> blocked out.
> 
> I don't have a strong opinion on this and I'm find with leaving at is
> but WX_OVERLAY does not make it clear that it's an OSX only feature only
> the wxOverlay is used in the graphics rendering.

WX_OVERLAY is AFAIK not OSX specific.
You could use it on any platform… although you probably wouldn’t want to do it 
(… welcome to the wonderful world of redraw problems currently only known
from OSX).

>> 
>> And yes, USE_WX_GRAPHICS_CONTEXT does build on non-OSX platforms. It's
>> slow but looks much better, I could see someone with a powerful machine
>> preferring it.
> 
> Wow!  This still compiles and works.  Way to go wxWidgets.  I added this
> a long time ago to improve our graphics quality but is was soooooo slow
> that I abandoned it.  On windows it used a library that MS has since
> abandoned so I considered it a dead end.  Who knew it would still be useful?
> 
>> 
>> 
>> On Sep 4, 2016 15:30, "Bernhard Stegmaier" <stegmaier@xxxxxxxxxxxxx <mailto:stegmaier@xxxxxxxxxxxxx>
>> <mailto:stegmaier@xxxxxxxxxxxxx <mailto:stegmaier@xxxxxxxxxxxxx>>> wrote:
>> 
>>    USE_WX_OVERLAY=ON is mandatory for OSX, there is currently no way around
>>    it to have default canvases in pcbnew and eeschema. It is needed as
>>    a workaround
>>    for not having a XOR draw mode, I don’t know if this is also the
>>    case on other
>>    platforms.
>> 
>>    USE_WX_GRAPHICS_CONTEXT=ON is optional on OSX, but some of the
>>    drawing optimisations in default canvases I did need it. I wouldn’t
>>    recommend to
>>    build with USE_WX_GRAPHICS_CONTEXT=OFF on OSX. Don’t know if this one
>>    is used on other platforms (the optimisations that need this could
>>    be also beneficial
>>    on other platforms, not only on OSX).
>> 
>>> On 04 Sep 2016, at 21:21, Wayne Stambaugh <stambaughw@xxxxxxxxx
>>    <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>> wrote:
>>> 
>>> Is there ever a reason to build OSX with USE_WX_OVERLAY=OFF or any
>>    other
>>> platform with USE_WX_OVERLAY=ON?  If not, maybe we should replace #if
>>> USE_WX_OVERLAY with #if OSX (or whatever it should be).
>>> 
>>> On 9/4/2016 3:18 PM, Simon Wells wrote:
>>>> i am pretty sure one of them at least is used on osx as its the
>>>> default set in cmake on osx
>>>> 
>>>> On Mon, Sep 5, 2016 at 7:15 AM, Wayne Stambaugh
>>    <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx> <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>> wrote:
>>>>> On 9/3/2016 6:49 AM, Nick Østergaard wrote:
>>>>>> 2016-09-01 19:02 GMT+02:00 Wayne Stambaugh
>>    <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx> <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>>:
>>>>>>> Sorry it took so long.  I took a look at Cirilo's changes an
>>    it looks
>>>>>>> good.  We still need a PKGBUILD file for msys2 with JPs patch
>>    so we
>>>>>>> don't have to build oce from source.
>>>>>>> 
>>>>>>> When I merge Cirilo's 3d-plugin branch and I had to create a
>>    commit
>>>>>>> message which made me the commit author.  This seems brain
>>    dead to me.
>>>>>>> Here is the new commit log entry:
>>>>>>> 
>>>>>>> commit 12c26047bffd54ccbb88348acfe7f653556450da
>>>>>>> Merge: fcedef8 b656a81
>>>>>>> Author: Wayne Stambaugh <stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>
>>    <mailto:stambaughw@xxxxxxxxx <mailto:stambaughw@xxxxxxxxx>>>
>>>>>>> Date:   Thu Sep 1 11:59:44 2016 -0400
>>>>>>> 
>>>>>>>   Merge remote-tracking branch 'cirilo/3d-plugin'
>>>>>>> 
>>>>>>> I really only want to show Cirilo's commit log entries as a
>>    new commit.
>>>>>>> WTF! I'm liking git less every time I use it.  What do I need
>>    to do to
>>>>>>> get the correct commit history from Cirlo's 3d-plugin branch?
>>>>>>> 
>>>>>>> On 9/1/2016 7:35 AM, Nick Østergaard wrote:
>>>>>>>> I would also like to see this merged (I have only tested the
>>    out of
>>>>>>>> tree plugin), but I suggest that the build switch is named
>>    something
>>>>>>>> prefixed with "KICAD_" like most other kicad specific build
>>    switches
>>>>>>>> are, except for the github plugin.  Maybe KICAD_USE_OCE.
>>>>>>> 
>>>>>>> For the sake of consistency, we probably should prefix
>>    everything with
>>>>>>> KICAD_.  I always liked the autotools --with- and --enable-
>>    prefixes for
>>>>>>> config options.  There is no consistency as far as cmake
>>    goes.  CMake
>>>>>>> should have adopted a standard prefix naming convention for
>>    options.
>>>>>>> 
>>>>>>>> 
>>>>>>>> And to that I will add that
>>    Documentation/development/compiling.md <http://compiling.md <http://compiling.md/>>
>>>>>>>> should be updated too to reflect the usage for that compile
>>    switch, :)
>>>>>>> 
>>>>>>> Yes, we need to be keeping these up to date.
>>>>>>> 
>>>>>> 
>>>>>> So would a patch to change USE_OCE to KICAD_USE_OCE with
>>    description
>>>>>> be accepted?
>>>>>> 
>>>>>> Aslo, I note that the recently introduced option
>>    USE_SCH_IO_MANAGER is
>>>>>> also missing a prefix and it not decribed in compiling.md
>>    <http://compiling.md <http://compiling.md/>> either.
>>>>>> Should that be changed too?
>>>>> 
>>>>> I may change it since I'm guessing none of the packages are be built
>>>>> with it enabled and that only myself and a few brave testers are
>>>>> probably the only ones using it.  Please keep in mind, this will be
>>>>> removed once the schematic I/O plugin manager is the default.
>>>>> 
>>>>>> 
>>>>>> There are also USE_WX_GRAPHICS_CONTEXT and USE_WX_OVERLAY, but
>>    I think
>>>>>> I will ignore those two.
>>>>> 
>>>>> These have been around for a long time.  I doubt
>>    USE_WX_GRAPHICS_CONTEXT
>>>>> even builds anymore.  Is USE_WX_OVERLAY still used by the OSX folks?
>>>>> Maybe we can just get rid of them.
>>>>> 
>>>>>> 
>>>>>>>> 
>>>>>>>> Nick
>>>>>>>> 
>>>>>>>> 2016-09-01 11:39 GMT+02:00 Tomasz Wlostowski
>>    <tomasz.wlostowski@xxxxxxx <mailto:tomasz.wlostowski@xxxxxxx> <mailto:tomasz.wlostowski@xxxxxxx <mailto:tomasz.wlostowski@xxxxxxx>>>:
>>>>>>>>> On 01.09.2016 10:30, Cirilo Bernardo wrote:
>>>>>>>>>> Just a ping to remind devs of a branch introducing the OCE
>>    plugin:
>>>>>>>>>> 
>>>>>>>>>> 
>>    https://code.launchpad.net/~cirilo-bernardo/kicad/+git/kicad-oce/+merge/303513 <https://code.launchpad.net/~cirilo-bernardo/kicad/+git/kicad-oce/+merge/303513>
>>    <https://code.launchpad.net/~cirilo-bernardo/kicad/+git/kicad-oce/+merge/303513 <https://code.launchpad.net/~cirilo-bernardo/kicad/+git/kicad-oce/+merge/303513>>
>>>>>>>>>> 
>>>>>>>>>> Since the OCE plugin code must be activated with -DUSE_OCE=ON
>>>>>>>>>> when configuring with CMake, I think it's safe to include
>>    this code. Having
>>>>>>>>>> the code in the main base will make it easier for other
>>    volunteers to help
>>>>>>>>>> sort out build and deployment issues of the OCE plugin on
>>    Windows and
>>>>>>>>>> OSX.
>>>>>>>>> 
>>>>>>>>> Hi Cirilo,
>>>>>>>>> 
>>>>>>>>> I fully agree to merge the OCE plug-in code. Users have been
>>    desperately
>>>>>>>>> asking for ages for STEP & IGES support in the 3d-viewer ;-)
>>>>>>>>> 
>>>>>>>>> @Wayne/@Orson: since this is optional, are there any
>>    obstacles for the
>>>>>>>>> merge?
>>>>>>>>> 
>>>>>>>>> Cheers,
>>>>>>>>> Tom
>>>>>>>>> 
>>>>>>>>> _______________________________________________
>>>>>>>>> Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <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 <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>>
>>>>>>>>> More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
>>    <https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>>
>>>>>>>> 
>>>>>>>> _______________________________________________
>>>>>>>> Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <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 <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>>
>>>>>>>> More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
>>    <https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>>
>>>>>>>> 
>>>>>>> 
>>>>>>> _______________________________________________
>>>>>>> Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <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 <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>>
>>>>>>> More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
>>    <https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>>
>>>>> 
>>>>> 
>>>>> _______________________________________________
>>>>> Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <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 <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>>
>>>>> More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
>>    <https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>>
>>> 
>>> 
>>> _______________________________________________
>>> Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <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 <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>>
>>> More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
>>    <https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>>
>> 
>> 
>>    _______________________________________________
>>    Mailing list: https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <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 <https://launchpad.net/~kicad-developers>
>>    <https://launchpad.net/~kicad-developers <https://launchpad.net/~kicad-developers>>
>>    More help   : https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>
>>    <https://help.launchpad.net/ListHelp <https://help.launchpad.net/ListHelp>>


References