[Feat] Some actions stuff... #913

antonymilne · 2024-12-02T20:48:36Z

Description

Screenshot

Notice

I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":
- I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
- I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
- I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
- I have not referenced individuals, products or companies in any commits, directly or indirectly.
- I have not added data or restricted code in any commits, directly or indirectly.

github-actions · 2024-12-02T20:49:51Z

View the example dashboards of the current commit live on PyCafe ☕ 🚀

Updated on: 2024-12-06 21:33:33 UTC
Commit: 21d705c

Link: vizro-core/examples/dev/

Link: vizro-core/examples/scratch_dev

Link: vizro-core/examples/visual-vocabulary/

Link: vizro-ai/examples/dashboard_ui/

This reverts commit 2ec1864.

…ther approach

… try another approach" This reverts commit cb27593.

…meters, filter_interaction arguments instead of **inputs

…l__ signature

for more information, see https://pre-commit.ci

…tion class like built-in ones or use shortcut @capture_new_action method. Current issues: * export_data and filter_interaction are public so still need to work using old method. This is why QA tests are failing. Need to make new_export_data that does it new way and keep old one for legacy? * have some good ideas for how to handle inputs of custom action but not sure how to handle outputs - ideas given in _action.py and app.py PR #911 was nearly there and first commits on this branch were tidying that, but then realised that pydantic model is probably better than CapturedCallable (see paper notes) so tried this new approach.

for more information, see https://pre-commit.ci

Test not using __name__ in continue

2ec1864

antonymilne and others added 10 commits December 4, 2024 10:43

Revert "Test not using __name__ in continue"

f11636a

This reverts commit 2ec1864.

Partial conversion to class-based action system but paused to try ano…

cb27593

…ther approach

Revert "Partial conversion to class-based action system but paused to…

6592f7c

… try another approach" This reverts commit cb27593.

Convert all built in actions to subclass NewAction pydantic model

b86f3d5

Move lots of stuff to _action.py and introduce explicit filters, para…

73502a6

…meters, filter_interaction arguments instead of **inputs

Provide filters etc. arguments only when requested in NewAction.__cal…

a9c5b51

…l__ signature

Merge branch 'main' into tidy/action-to-class-2

7d0f780

[pre-commit.ci] auto fixes from pre-commit.com hooks

c7d55ad

for more information, see https://pre-commit.ci

[pre-commit.ci] auto fixes from pre-commit.com hooks

21d705c

for more information, see https://pre-commit.ci

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] Some actions stuff... #913

[Feat] Some actions stuff... #913

antonymilne commented Dec 2, 2024

github-actions bot commented Dec 2, 2024 •

edited

Loading

[Feat] Some actions stuff... #913

Are you sure you want to change the base?

[Feat] Some actions stuff... #913

Conversation

antonymilne commented Dec 2, 2024

Description

Screenshot

Notice

github-actions bot commented Dec 2, 2024 • edited Loading

View the example dashboards of the current commit live on PyCafe ☕ 🚀

github-actions bot commented Dec 2, 2024 •

edited

Loading