kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #27114
Re: Current state of ActionPlugin
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Jean-Samuel Reynaud <js.reynaud@xxxxxxxxx>
-
Date:
Tue, 10 Jan 2017 16:25:10 +0100
-
In-reply-to:
<CAJjCH9ryyVEiRKQi-6Wt-um=qqNX_GRZixR1VsgJLqziekhfvg@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0
Hi,
Just from my point of view:
We use some python plugin currently and it's not really easy to open
each time python console (and remember plugins name, call mode...).
I'm agree with you: During devs of scripts it's crashing and my devs
have to find the rigth way to process.
But this kind of feature can increase number of users/devs of this kind
of scripts and raise crash case easily.
Regards,
Le 10/01/2017 à 16:00, Chris Pavlina a écrit :
> Frankly I don't think we should be encouraging users to use the Python
> API until it is significantly improved. The issues JP raises make the
> API completely unsuitable for plugin use. It's opaque and no users have
> any idea how to write robust Python plugins given the current state of
> the API.
>
> On Jan 10, 2017 09:55, "jp charras" <jp.charras@xxxxxxxxxx
> <mailto:jp.charras@xxxxxxxxxx>> wrote:
>
> Le 10/01/2017 à 15:18, Maciej Sumiński a écrit :
> > Hi Jean-Samuel,
> >
> > I think your patch will facilitate use of 3rd party python plugins, as
> > the current way of executing commands from the Python shell is not
> quite
> > user friendly. I vote for merging the patch, but we need to fix some
> > code formatting issues first.
> >
> > Also, I wonder if it would be the right thing to group menu entries by
> > their categories to create submenus. It could be a simple way to
> > organize the actions provided by python plugins. Just an idea.
> >
> > Another question is about the following lines:
> > + if( IsGalCanvasActive() )
> > + {
> > + UseGalCanvas( GetGalCanvas() );
> > + }
> >
> > What is exactly the goal here? If it is only about refreshing the
> > canvas, then a simple Refresh() call should fix the problem.
> >
> > Regards,
> > Orson
> >
>
> This is a good work.
> Thanks Jean-Samuel.
>
> However, before merging we have to take care of issues which can
> easily crash Pcbnew:
> Because a python script can modify the board outside the control of
> pcbnew edit functions, I am
> thinking problems will arise with undo/redo lists (invalid
> pointers), ratsnest data and certainly a
> few other things.
>
> Especially for scripts which add or delete items, or change
> connectivity.
>
> Of course, undo/redo lists are alway incorrect after running such
> scripts.
> This is already a known issue for scripts that are run from the
> Pcbnew python console.
>
> --
> Jean-Pierre CHARRAS
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> Unsubscribe : https://launchpad.net/~kicad-developers
> <https://launchpad.net/~kicad-developers>
> More help : https://help.launchpad.net/ListHelp
> <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
-
Current state of ActionPlugin
From: Jean-Samuel Reynaud, 2017-01-03
-
Re: Current state of ActionPlugin
From: Wayne Stambaugh, 2017-01-04
-
Re: Current state of ActionPlugin
From: Jean-Samuel Reynaud, 2017-01-04
-
Re: Current state of ActionPlugin
From: Wayne Stambaugh, 2017-01-04
-
Re: Current state of ActionPlugin
From: Jean-Samuel Reynaud, 2017-01-09
-
Re: Current state of ActionPlugin
From: Nick Østergaard, 2017-01-09
-
Re: Current state of ActionPlugin
From: Jean-Samuel Reynaud, 2017-01-10
-
Re: Current state of ActionPlugin
From: Maciej Sumiński, 2017-01-10
-
Re: Current state of ActionPlugin
From: jp charras, 2017-01-10
-
Re: Current state of ActionPlugin
From: Chris Pavlina, 2017-01-10