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

fixing walk order to resolve priority in multi-sink pipelines #120

Merged
merged 3 commits into from
Nov 13, 2024

add test for stream submission ordering logic

8c2e772
Select commit
Loading
Failed to load commit list.
Refresh
Merged

fixing walk order to resolve priority in multi-sink pipelines #120

add test for stream submission ordering logic
8c2e772
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Nov 11, 2024 in 1s

2.56% of diff hit (target 100.00%)

View this Pull Request on Codecov

2.56% of diff hit (target 100.00%)

Annotations

Check warning on line 108 in dplutils/pipeline/graph.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/graph.py#L106-L108 

Added lines #L106 - L108 were not covered by tests

Check warning on line 118 in dplutils/pipeline/graph.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/graph.py#L112-L118 

Added lines #L112 - L118 were not covered by tests

Check warning on line 123 in dplutils/pipeline/graph.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/graph.py#L121-L123 

Added lines #L121 - L123 were not covered by tests

Check warning on line 185 in dplutils/pipeline/stream.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/stream.py#L184-L185 

Added lines #L184 - L185 were not covered by tests

Check warning on line 189 in dplutils/pipeline/stream.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/stream.py#L188-L189 

Added lines #L188 - L189 were not covered by tests

Check warning on line 194 in dplutils/pipeline/stream.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/stream.py#L191-L194 

Added lines #L191 - L194 were not covered by tests

Check warning on line 200 in dplutils/pipeline/stream.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/stream.py#L196-L200 

Added lines #L196 - L200 were not covered by tests

Check warning on line 219 in dplutils/pipeline/stream.py

See this annotation in the file changed.

@codecov codecov / codecov/patch

dplutils/pipeline/stream.py#L208-L219 

Added lines #L208 - L219 were not covered by tests
View more details on Codecov