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

[FEAT] Propagate configs to Ray remote functions #1707

Merged
merged 10 commits into from
Dec 8, 2023

Update RayRunnerConfig to be frozen dataclass

21bec7c
Select commit
Loading
Failed to load commit list.
Merged

[FEAT] Propagate configs to Ray remote functions #1707

Update RayRunnerConfig to be frozen dataclass
21bec7c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 8, 2023 in 0s

81.94% of diff hit (target 85.11%)

View this Pull Request on Codecov

81.94% of diff hit (target 85.11%)

Annotations

Check warning on line 161 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L160-L161

Added lines #L160 - L161 were not covered by tests

Check warning on line 166 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L166

Added line #L166 was not covered by tests

Check warning on line 169 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L169

Added line #L169 was not covered by tests

Check warning on line 174 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L173-L174

Added lines #L173 - L174 were not covered by tests

Check warning on line 186 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L185-L186

Added lines #L185 - L186 were not covered by tests

Check warning on line 191 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L189-L191

Added lines #L189 - L191 were not covered by tests

Check warning on line 213 in daft/context.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/context.py#L213

Added line #L213 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

daft/runners/ray_runner.py#L147

Added line #L147 was not covered by tests