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

Kwarg logic for plugin interface needs fixing #444

Open
brifordwylie opened this issue Apr 3, 2024 · 0 comments
Open

Kwarg logic for plugin interface needs fixing #444

brifordwylie opened this issue Apr 3, 2024 · 0 comments
Assignees
Labels
bug Bug to be Fixed dash Dash/Plotly Work plugins Plugin Architecture and Development

Comments

@brifordwylie
Copy link
Member

So the addition of kwargs is a nice bit of flexibility so that plugins can can in additional information, but looks like we still have a wrinkle with the implementation. Works fine for plugins that take an inference_run arg.. but crashing when they don't take that arg.

Screenshot 2024-04-03 at 4 35 02 PM

Related to: #443

@brifordwylie brifordwylie added bug Bug to be Fixed dash Dash/Plotly Work plugins Plugin Architecture and Development labels Apr 3, 2024
@brifordwylie brifordwylie added this to the SageWorks 0.5.0 milestone Apr 3, 2024
@brifordwylie brifordwylie self-assigned this Apr 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug to be Fixed dash Dash/Plotly Work plugins Plugin Architecture and Development
Projects
Status: Backlog
Development

No branches or pull requests

1 participant