-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add 'Plugin manager' section to 'napari architecture guide' #321
Comments
Ahh @lucyleeow when you mentioned this, I was thinking about the plugin installation manager but I see now that you mean the internal plugin manager. It would probably be nice for us to disambiguate this in the docs also 😆 . |
Information that would be good to include:
|
Is this in the context of testing? Either way, I think it would be useful to have a section on testing plugins in napari, Dynamic Plugin and how to add contributions via the
This has tripped me up too. Naming is hard. |
Noticed in #370 that we do not have info on (I know its mentioned above in #321 (comment) but just highlighting again) |
Oh wow, yes, that's confusing, I had to read that sentence a few times to actually parse it 😬
Are there better terms we can use? Should we exclusively refer to the plugin registration library as the "plugin engine" instead of manager? |
Yes there should be different terms! npe1 and npe2 (the plugin engines) both have a class called Maybe Plugin installation manager GUI, could be "plugin manager widget"? or "plugin installation manager"? |
Mmm yeah this is a sad state of affairs 😆 . I think we should rename the internal plugin manager(s) rather than the plugin installation manager, since that is a public facing package and also is named in the menus, while the internal plugin managers aren't. I would be in favour of renaming to something like |
Yeah good point. I just don't think of the GUI when I hear 'plugin manager' and then have to remind myself it's the GUI thing.
Any potential with confusion that this would be an app-model thing because app-model has the CommandRegistry, MenuRegistry...etc ? |
What a can of worms... the plugin manager used to install/etc. plugins isn't live yet--0.4.19 uses the old UI--so in principle we could still change that if it would make life easier... |
🧰 Task
We are lacking documentation on the npe2 Plugin manager. The 'plugins' section talks about building plugins, and information about the plugin manager would not be relevant here. Plugin manager information is more relevant to napari developers. Currently there is some mention of it in NAP 2.
E.g., information about the plugin manager command registry and indexing would fit here, see: napari/npe2#314
cc @jaimergp and @DragaDoncila who may have the knowledge to add such a section.
The text was updated successfully, but these errors were encountered: