[Feat] Implement AgGrid model - Githubissues

mckinsey / vizro

Vizro is a toolkit for creating modular data visualization applications.

https://vizro.readthedocs.io/en/stable/

Apache License 2.0

2.49k stars 112 forks source link

[Feat] Implement AgGrid model #289

Closed maxschulz-COL closed 5 months ago

maxschulz-COL commented 6 months ago

Description

This PR is an iteration of discussion in https://github.com/mckinsey/vizro/pull/260. Here we decided to implement one model per Callable type.

Example Dashboard contains main features
Formatting of Data inspired by @AnnMarieW and the new features in AGGrid 31.0.0
no styling applied
no tests
no docs

Styling, docs and tests will be in separate PRs.

Notable activated features

Columns are resizeable
Columns are sortable
Columns are filterable (with an apply and reset button)
Activated pagination
Deactivated Row animation

Cell formatting

Defined 4 additional data types for easy column formatting - will be explained in the docs: dollar, euro, percent, numeric - see third page of dev example and Screenshot
Filters adjust to cell content type: numeric, date, or string/other
(to be discussed) Auto-format any detected pd.datetime column into a valid dateString (required by AGGrid to function for dates out of the box)

Screenshot

Notice

[x] 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.

petar-qb commented 5 months ago

The initial bug is solved with the latest changes https://github.com/mckinsey/vizro/pull/289/commits/ec85e0c0dfcd5693d576e5fcc531f8bef88c3ca2 you pushed. Awesome work Max! 🤖