Skip to content
Snippets Groups Projects

CAPO handling for NAASC submission

Merged Charlotte Hausman requested to merge relocatable_workflows into main
3 unresolved threads

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
21 21 """
22 22
23 23 def __init__(self, wf_request: WorkflowRequestIF):
24 self.profile = CapoConfig().profile
25 24 self.request = wf_request
25 self.profile = self.set_profile()
26 26 self.views = ViewWorkflowCapoParameters(wf_request)
27 27 self.dir_path = Path(self.request.results_dir)
28 28
29 def set_profile(self):
  • 34 system = current.split("-")[1]
    35 remote_profile = "naasc-" + system
    36 return remote_profile
    37
    38 return current
    39
    29 40 def is_remote_workflow(self) -> bool:
    30 41 """
    31 42 Determine if the provided request is for a workflow that can run remotely
    32 43
    33 44 :return: boolean representing if this workflow request is for a remote executing workflow
    34 45 """
    35 return RemoteWorkflows.is_remote(wf_name=self.request.workflow_name)
    46 parameters = self.request.argument
    47 remote_types = RemoteProcessingSites.values()
    48 if any([key in remote_types for key in parameters]) and RemoteWorkflows.is_remote(
  • 40
    41 class RemoteProcessingSites(Enum):
    42 """
    43 Enum specifying which remote sites are available
    44 """
    45
    46 NAASC = "remote-cv"
    47 NMT = "remote"
    48
    49 @classmethod
    50 def values(cls):
    51 return cls._value2member_map_
    52
    53 @classmethod
    54 def is_remote(cls, remote_type: str):
    55 return remote_type in cls.values()
  • added 1 commit

    Compare with previous version

  • Can some of you look this over so we can start test runs in CV?

  • Daniel Lyons approved this merge request

    approved this merge request

  • Nathan Hertz approved this merge request

    approved this merge request

  • Janet Goldstein approved this merge request

    approved this merge request

  • Please register or sign in to reply
    Loading