Skip to content

partition_by with complex signals - includes agg and group_by #1215

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

Merged
merged 35 commits into from
Jul 12, 2025

Add more tests

09b8a6e
Select commit
Loading
Failed to load commit list.
Merged

partition_by with complex signals - includes agg and group_by #1215

Add more tests
09b8a6e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jul 12, 2025 in 0s

90.42% of diff hit (target 88.68%)

View this Pull Request on Codecov

90.42% of diff hit (target 88.68%)

Annotations

Check warning on line 19 in src/datachain/lib/convert/sql_to_python.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/datachain/lib/convert/sql_to_python.py#L19 

Added line #L19 was not covered by tests

Check warning on line 957 in src/datachain/lib/dc/datachain.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/datachain/lib/dc/datachain.py#L957 

Added line #L957 was not covered by tests

Check warning on line 1038 in src/datachain/query/dataset.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/datachain/query/dataset.py#L1037-L1038 

Added lines #L1037 - L1038 were not covered by tests
View more details on Codecov