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

chore: Build progress bar only on first update #3626

Merged
merged 4 commits into from
Dec 21, 2024

still clear it, but up the update interval

4ba1200
Select commit
Loading
Failed to load commit list.
Merged

chore: Build progress bar only on first update #3626

still clear it, but up the update interval
4ba1200
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 19, 2024 in 0s

0.00% of diff hit (target 77.80%)

View this Pull Request on Codecov

0.00% of diff hit (target 77.80%)

Annotations

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/runners/progress_bar.py#L117-L118

Added lines #L117 - L118 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/runners/progress_bar.py#L121-L123

Added lines #L121 - L123 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/runners/progress_bar.py#L127-L131

Added lines #L127 - L131 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/runners/progress_bar.py#L139

Added line #L139 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/runners/progress_bar.py#L143-L145

Added lines #L143 - L145 were not covered by tests