Skip to content

Commit

Permalink
Remove workgraph-dev.py dev file.
Browse files Browse the repository at this point in the history
  • Loading branch information
GeigerJ2 committed Dec 19, 2024
1 parent 3994ccc commit 4e99d2e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 43 deletions.
4 changes: 1 addition & 3 deletions src/sirocco/workgraph.py
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,7 @@ def _add_aiida_task_node(self, task: graph_items.Task):
msg = f"The command is None of task {task}."
raise ValueError(msg)

# TODO: Add proper resolving of code/command to installed/portable code, etc., to think about
command = task.command

# ? Source file
Expand All @@ -187,7 +188,6 @@ def _add_aiida_task_node(self, task: graph_items.Task):
# we create the task. Instead, they are being updated via the WG internals when linking inputs/outputs to
# tasks
argument_list = task.cli_argument.split()
# breakpoint()
workgraph_task = self._workgraph.tasks.new(
"ShellJob",
name=label,
Expand Down Expand Up @@ -260,8 +260,6 @@ def _link_input_to_task(self, task: graph_items.Task, input_: graph_items.Data):
msg = f"Workgraph task {workgraph_task.name!r} did not initialize arguments nodes in the workgraph before linking. This is a bug in the code, please contact devevlopers."
raise ValueError(msg)
# TODO think about that the yaml file should have aiida valid labels
# if (arg_option := task.input_arg_options.get(input_.name, None)) is not None:
# workgraph_task_arguments.value.append(f"{arg_option}")

# Avoid appending the same argument twice
argument_placeholder = f"{{{input_label}}}"
Expand Down
40 changes: 0 additions & 40 deletions workgraph-dev.py

This file was deleted.

0 comments on commit 4e99d2e

Please sign in to comment.