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

[FEAT] Enable Progress Bars for PyRunner and RayRunner #1609

Merged

samster25 merged 13 commits into main from sammy/progress-bar-prototype-pyrunner

Nov 17, 2023

Merged

[FEAT] Enable Progress Bars for PyRunner and RayRunner #1609

dont refresh ray progress bar and set update freq to 1 second

Codecov / codecov/patch succeeded Nov 17, 2023 in 1s

88.17% of diff hit (target 84.59%)

View this Pull Request on Codecov

88.17% of diff hit (target 84.59%)

Annotations

Check warning on line 758 in daft/expressions/expressions.py

codecov / codecov/patch

daft/expressions/expressions.py#L758

Added line #L758 was not covered by tests

Check warning on line 25 in daft/runners/progress_bar.py

codecov / codecov/patch

daft/runners/progress_bar.py#L25

Added line #L25 was not covered by tests

Check warning on line 33 in daft/runners/progress_bar.py

codecov / codecov/patch

daft/runners/progress_bar.py#L33

Added line #L33 was not covered by tests

Check warning on line 36 in daft/runners/progress_bar.py

codecov / codecov/patch

daft/runners/progress_bar.py#L35-L36

Added lines #L35 - L36 were not covered by tests

Check warning on line 39 in daft/runners/progress_bar.py

codecov / codecov/patch

daft/runners/progress_bar.py#L38-L39

Added lines #L38 - L39 were not covered by tests

Check warning on line 52 in daft/runners/progress_bar.py

codecov / codecov/patch

daft/runners/progress_bar.py#L52

Added line #L52 was not covered by tests

Check warning on line 57 in daft/runners/progress_bar.py

codecov / codecov/patch

daft/runners/progress_bar.py#L57

Added line #L57 was not covered by tests

Check warning on line 587 in daft/runners/ray_runner.py

codecov / codecov/patch

daft/runners/ray_runner.py#L587

Added line #L587 was not covered by tests

Check warning on line 641 in daft/runners/ray_runner.py

codecov / codecov/patch

daft/runners/ray_runner.py#L641

Added line #L641 was not covered by tests

View more details on Codecov