New steps decorator #1062
Merged
New steps decorator #1062
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
elliotgunton
added
type:enhancement
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
2828d73
to
196e10b
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
2493d5d
to
ac0234c
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
196e10b
to
f160b78
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
ac0234c
to
2c4bda3
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
f160b78
to
4b4e283
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
2c4bda3
to
343cab8
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
2 times, most recently
from
b59c538
to
dc945ca
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
367f499
to
463db8d
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
dc945ca
to
cb235a3
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
2 times, most recently
from
cd51680
to
6a2ca54
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
cb235a3
to
02f5fea
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
63cd710
to
8de9d3b
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
5 times, most recently
from
311ec06
to
8e1e836
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
5b01fb8
to
72d6534
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
8e1e836
to
104999b
Compare
elliotgunton
force-pushed
the
new-dag-decorator
branch
from
72d6534
to
7cfb6b8
Compare
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
104999b
to
f6f3a48
Compare
samj1912
reviewed
May 15, 2024
* Extract out common code from dag decorator Signed-off-by: Elliot Gunton <egunton@bloomberg.net>
Signed-off-by: Elliot Gunton <egunton@bloomberg.net>
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
3ce3d10
to
ff6eac6
Compare
Signed-off-by: Elliot Gunton <egunton@bloomberg.net>
elliotgunton
force-pushed
the
new-steps-decorator
branch
from
ff6eac6
to
9ef804f
Compare
samj1912
approved these changes
May 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Checklist
Description of PR
PR refactors common code from the dag decorator, and adds the new
parallelfunction to create aParallelcontext at declaration time, or a dummy context to allow local execution.