Skip to content

[QA] Background waiter in tests #1375

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open

[QA] Background waiter in tests #1375

wants to merge 61 commits into from

Conversation

l0uden
Copy link
Contributor

@l0uden l0uden commented Sep 4, 2025 •
edited
Loading

Description

Detailed description behind the reason of the waiter can be found in this task

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.

l0uden added 30 commits April 1, 2025 14:32
@l0uden
data frame parameter test
8795181
@l0uden
Revert "data frame parameter test"
69f17b2 
This reverts commit 8795181.
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
3134da0
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b71e210
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
7f01cae
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
e3c7f93
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
008cb72
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
6305456
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
ace4b5b
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b11ed48
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
ae42e28
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b8711b5
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
38cebdc
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
4d3464f
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
0cb855a
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
55591e6
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
2bb768e
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
1389c95
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b10cc9c
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
32287e9
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
242b101
@l0uden
test extra parameter for graph
5ca5573
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
18682c4
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
bd2b546
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
31b8775
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
0f46334
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
a5f9730
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b8e3790
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
6722cd4
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
3ab01a6
l0uden added 13 commits July 28, 2025 14:10
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
a5e4eae
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b03889f
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
cb60a21
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
b0d68b3
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
8c223f9
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
53eebac
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
96f7de8
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
fcfb46a
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
f56a9bf
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
895b1a2
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro
662c6e7
@l0uden
background waiter in thread
0f0afbf
@l0uden
Merge branch 'main' of https://github.com/mckinsey/vizro into qa/back…
7e1d1b8 
…ground_waiter_in_tests
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025 •
edited
Loading

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

Updated on: 2025-09-12 10:50:13 UTC
Commit: 0fb2cf2

Compare the examples using the commit's wheel file vs the latest released version:

vizro-core/examples/scratch_dev

View with commit's wheel vs View with latest release

vizro-core/examples/dev/

View with commit's wheel vs View with latest release

vizro-core/examples/visual-vocabulary/

View with commit's wheel vs View with latest release

vizro-core/examples/tutorial/

View with commit's wheel vs View with latest release

@l0uden l0uden requested a review from petar-qb September 5, 2025 11:39
@l0uden l0uden marked this pull request as ready for review September 5, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@petar-qb petar-qb Awaiting requested review from petar-qb

@Joseph-Perkins Joseph-Perkins Awaiting requested review from Joseph-Perkins Joseph-Perkins is a code owner

@antonymilne antonymilne Awaiting requested review from antonymilne antonymilne is a code owner

@huong-li-nguyen huong-li-nguyen Awaiting requested review from huong-li-nguyen huong-li-nguyen is a code owner

@maxschulz-COL maxschulz-COL Awaiting requested review from maxschulz-COL maxschulz-COL is a code owner

@lingyielia lingyielia Awaiting requested review from lingyielia lingyielia is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@l0uden l0uden

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@l0uden