Skip to content
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

WIP - pluggy plugins pluggy wuggy #2111

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

WIP - pluggy plugins pluggy wuggy #2111

wants to merge 16 commits into from

Conversation

dgtlmoon
Copy link
Owner

@dgtlmoon dgtlmoon commented Jan 15, 2024

image

plugin ideas

  • whois
  • base model pytorch to pick out stock/not in stock
  • base model pytorch for extracting headings+links
  • append all http headers to start of the change text

todo

  • "plugins" page to enable/disable
  • testing
  • documentation of hooks, diagram of the flow (fetch backend, processors, hooks etc)
  • ability to add [enabled] or other forms to edit watch and edit global
  • ability to have a config panel per plugin
  • ability to extend the main overview list page with other data (custom column)
  • events on change detected, change detect started, site added, etc (for example, automatically identify a filter)

other

  • some text icon/bubble in the watch overview to show which plugin its running with

elif processor.startswith('plugin_processor_'):
from .flask_app import get_plugin_manager
pm = get_plugin_manager()
x = pm.hook.processor_call(processor_name=processor, datastore=self.datastore, watch_uuid=uuid)
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test this when running multiple different plugins :/

@dgtlmoon dgtlmoon changed the title WIP - pluggy 2 WIP - pluggy plugins Jan 19, 2024
@dgtlmoon dgtlmoon added the WIP label Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant