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.

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

refactor: port DaftContext to rust side #3767

Merged
merged 25 commits into from
Feb 6, 2025

make state more private

144ad60
Select commit
Loading
Failed to load commit list.
Merged

refactor: port DaftContext to rust side #3767

make state more private
144ad60
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Feb 6, 2025 in 1s

84.18% of diff hit (target 75.88%)

View this Pull Request on Codecov

84.18% of diff hit (target 75.88%)

Annotations

Check warning on line 46 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L46 

Added line #L46 was not covered by tests

Check warning on line 133 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L133 

Added line #L133 was not covered by tests

Check warning on line 138 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L138 

Added line #L138 was not covered by tests

Check warning on line 71 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L70-L71 

Added lines #L70 - L71 were not covered by tests

Check warning on line 110 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L110 

Added line #L110 was not covered by tests

Check warning on line 118 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L118 

Added line #L118 was not covered by tests

Check warning on line 129 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L127-L129 

Added lines #L127 - L129 were not covered by tests

Check warning on line 134 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L132-L134 

Added lines #L132 - L134 were not covered by tests

Check warning on line 139 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L137-L139 

Added lines #L137 - L139 were not covered by tests

Check warning on line 145 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L144-L145 

Added lines #L144 - L145 were not covered by tests

Check warning on line 149 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L148-L149 

Added lines #L148 - L149 were not covered by tests

Check warning on line 153 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L152-L153 

Added lines #L152 - L153 were not covered by tests

Check warning on line 157 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L156-L157 

Added lines #L156 - L157 were not covered by tests

Check warning on line 161 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L160-L161 

Added lines #L160 - L161 were not covered by tests

Check warning on line 190 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L189-L190 

Added lines #L189 - L190 were not covered by tests

Check warning on line 218 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L213-L218 

Added lines #L213 - L218 were not covered by tests

Check warning on line 235 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L234-L235 

Added lines #L234 - L235 were not covered by tests

Check warning on line 252 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L251-L252 

Added lines #L251 - L252 were not covered by tests

Check warning on line 268 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L267-L268 

Added lines #L267 - L268 were not covered by tests

Check warning on line 270 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L270 

Added line #L270 was not covered by tests

Check warning on line 338 in src/daft-context/src/lib.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/lib.rs#L337-L338 

Added lines #L337 - L338 were not covered by tests

Check warning on line 8 in src/daft-context/src/python.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/python.rs#L8 

Added line #L8 was not covered by tests

Check warning on line 21 in src/daft-context/src/python.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/python.rs#L19-L21 

Added lines #L19 - L21 were not covered by tests

Check warning on line 31 in src/daft-context/src/python.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/python.rs#L27-L31 

Added lines #L27 - L31 were not covered by tests

Check warning on line 82 in src/daft-context/src/python.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/daft-context/src/python.rs#L79-L82 

Added lines #L79 - L82 were not covered by tests
View more details on Codecov