Replace `inputs_complete` by `force_start_node` to forcefully mark nodes as start nodes
Merged
Replace `inputs_complete` by `force_start_node` to forcefully mark nodes as start nodes
start-node
into
main
3 unresolved threads
3 unresolved threads
Fix #58 (closed)
Related to ewoksppf#16
Merge request reports
Activity
added 3 commits
-
4f8c67be...375fee02 - 2 commits from branch
main
- 0aaad512 - Replace `input_complete` by `start_node` to explicitly mark nodes as start nodes
-
4f8c67be...375fee02 - 2 commits from branch
added 1 commit
- 3f31bbd5 - Replace `inputs_complete` by `start_node` to explicitly mark nodes as start nodes
151 151 { 152 152 "default_inputs": [{"name":"a", "value":1}] 153 153 } 154 * *inputs_complete* (optional): set to `True` when the default inputs cover all required inputs 155 (used for *method*, *script* and *notebook* as their required inputs are unknown) 154 * *start_node* (optional): when set to `True`, the node will be explicitly defined as a start node i.e. a node that should be executed before all others. 155 (To be used as an escape hatch when the graph analysis fails to correctly assert the start nodes). changed this line in version 5 of the diff
- src/ewokscore/tests/test_start_node.py 0 → 100644
1 import pytest 2 import os.path 3 from ewokscore import Task 4 from ewoks import execute_graph 5 6 7 def test_explicit_start_node_on_self_triggering_node(tmpdir): changed this line in version 4 of the diff
added 1 commit
- 5ca0ceaf - Replace `inputs_complete` by `start_node` to explicitly mark nodes as start nodes
requested review from @denolf
mentioned in merge request !228 (closed)
mentioned in merge request ewoksppf!101
297 296 return False 298 297 299 298 299 def node_is_start_node(graph: networkx.DiGraph, node_id: NodeIdType) -> bool: 300 node = graph.nodes[node_id] 301 if node.get("start_node", False): changed this line in version 5 of the diff
assigned to @loic.huder
added 1 commit
- 4676d81f - Replace `inputs_complete` by `force_start_node` to forcefully mark nodes as start nodes
mentioned in commit bc7faedb
mentioned in issue ewoksppf#16
Please register or sign in to reply