-
Notifications
You must be signed in to change notification settings - Fork 17
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
Let plugins instance load a new plugin (url) without overwriting the ones that are already loaded. #151
Comments
@edreisMD, I have started working on addressing this issue in PR #151. My plan is to add an Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.plugnplai/plugnplai/plugins.py Lines 257 to 546 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 210 to 514 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 227 to 545 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 16 to 36 in b4b09ff
Lines 76 to 119 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 545 to 546 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 73 to 130 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 1 to 16 in b4b09ff
Lines 34 to 45 in b4b09ff
Lines 143 to 174 in b4b09ff
I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
I'm sorry, but it looks like an error has occured. Try removing and re-adding the sweep label. I'll try again in a minute. If this error persists contact [email protected]. I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
@edreisMD, I have started working on implementing this feature in the Plugins class. My plan is to add an install_plugin() method that will load a new plugin's manifest and OpenAPI spec, create a PluginObject instance for it, and add that to the installed_plugins dictionary. This will allow installing new plugins without overwriting existing ones. Give me a minute! Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.plugnplai/plugnplai/plugins.py Lines 257 to 546 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 210 to 514 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 227 to 545 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 16 to 36 in b4b09ff
Lines 76 to 119 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 545 to 546 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 73 to 130 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 1 to 16 in b4b09ff
Lines 34 to 45 in b4b09ff
Lines 143 to 174 in b4b09ff
I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
I'm sorry, but it looks like an error has occured. Try removing and re-adding the sweep label. I'll try again in a minute. If this error persists contact [email protected]. I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
Hey @edreisMD! I've started working on adding the ability to install a new plugin without overwriting existing ones. My plan is to add an Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.plugnplai/plugnplai/plugins.py Lines 257 to 546 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 210 to 514 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 227 to 545 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 16 to 36 in b4b09ff
Lines 76 to 119 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 545 to 546 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 73 to 130 in b4b09ff
Lines 34 to 45 in b4b09ff
Lines 143 to 174 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 57 to 97 in b4b09ff
I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
@edreisMD, I've started working on implementing this feature in the Plugins class. My plan is to add an install_plugin() method that will load a single plugin's manifest and spec, create a PluginObject, and add that to the installed_plugins dictionary. This will allow installing new plugins without overwriting existing ones. Give me a minute! Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.plugnplai/plugnplai/plugins.py Lines 257 to 546 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 210 to 514 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 227 to 545 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 16 to 36 in b4b09ff
Lines 76 to 119 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 545 to 546 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 73 to 130 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 1 to 16 in b4b09ff
Lines 34 to 45 in b4b09ff
Lines 143 to 174 in b4b09ff
I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
I'm sorry, but it looks like an error has occured. Try removing and re-adding the sweep label. I'll try again in a minute. If this error persists contact [email protected]. I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
@edreisMD, I've started working on implementing the ability to install a new plugin from a URL without overwriting existing plugins. My plan is to add an Some code snippets I looked at (click to expand). If some file is missing from here, you can mention the path in the ticket description.plugnplai/plugnplai/plugins.py Lines 257 to 546 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 210 to 514 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 227 to 545 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 16 to 36 in b4b09ff
Lines 76 to 119 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 545 to 546 in b4b09ff
plugnplai/plugnplai/embeddings.py Lines 73 to 130 in b4b09ff
Lines 34 to 45 in b4b09ff
Lines 143 to 174 in b4b09ff
plugnplai/plugnplai/plugins.py Lines 57 to 97 in b4b09ff
I'm a bot that handles simple bugs and feature requests but I might make mistakes. Please be kind! |
build a function on the plugins object to install a new plugin based on the url without forgetting the previous installed.
The text was updated successfully, but these errors were encountered: