Project

General

Profile

Actions

Idea #10112

closed

[Crunch2][Workbench]workflow#show page (similar to pipeline_template#show page)

Added by Radhika Chippada over 7 years ago. Updated almost 7 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Radhika Chippada
Category:
-
Target version:
Start date:
05/09/2017
Due date:
Story points:
1.0
Release:
Release relationship:
Auto

Description

Include editable name and description field.
Also should include a "Run this workflow" button.


Files

workflow.jpg (61.4 KB) workflow.jpg Radhika Chippada, 05/09/2017 03:03 PM

Subtasks 1 (0 open1 closed)

Task #11472: Review 10112-workflow-showResolvedRadhika Chippada05/09/2017Actions
Actions #1

Updated by Peter Amstutz about 7 years ago

  • Description updated (diff)
Actions #2

Updated by Tom Morris about 7 years ago

  • Target version set to 2017-04-26 sprint
Actions #3

Updated by Tom Morris about 7 years ago

  • Description updated (diff)
  • Story points set to 1.0
Actions #4

Updated by Lucas Di Pentima about 7 years ago

  • Assigned To set to Lucas Di Pentima
Actions #5

Updated by Radhika Chippada almost 7 years ago

  • Status changed from New to In Progress
  • Assigned To changed from Lucas Di Pentima to Radhika Chippada
Actions #6

Updated by Radhika Chippada almost 7 years ago

  • Target version changed from 2017-04-26 sprint to 2017-05-10 sprint
Actions #7

Updated by Bryan Cosca almost 7 years ago

Showing these would be helpful:

baseCommand: [bwa, mem]
arguments: - {prefix: -t, valueFrom: $(runtime.cores)} - {prefix: -R, valueFrom: "@RG\tID:$(inputs.group_id)\tPL:$(inputs.PL)\tSM:$(inputs.sample_id)"}

(baseCommand before arguments)

hints: - {class: DockerRequirement, dockerImageId: biodckr/bwa, dockerPull: biodckr/bwa}

The DockerImageId should link to the collection of the docker image.

inputs:

outputs: if possible

Actions #8

Updated by Radhika Chippada almost 7 years ago

Actions #9

Updated by Radhika Chippada almost 7 years ago

Branch 10112-workflow-show @ 4c1d34e

Using the partial show_text_with_locators (from #10645), displays baseCommand, arguments, hints, inputs, outputs in workflow Definition tab.

Test run @ https://ci.curoverse.com/job/developer-run-tests/281/

Actions #10

Updated by Lucas Di Pentima almost 7 years ago

A couple comments:

  • DockerImageId is not being rendered as a collection link as suggested on note-7, is this possible?
  • Will the workflow definition have additional data that we would like to show? Maybe we can have a generic way of showing the YAML so that the partial doesn’t need to be updated when using a wf_def with new fields?
Actions #11

Updated by Radhika Chippada almost 7 years ago

DockerImageId is not being rendered as a collection link as suggested on note-7, is this possible?

Peter said it is quite involved and for now we just display the text

Will the workflow definition have additional data that we would like to show? Maybe we can have a generic way of showing the YAML so that the partial doesn’t need to be updated when using a wf_def with new fields?

For now this would do as Bryan suggested. Other fields may or may not fit the same display strategy (for example, uuids etc).

Actions #12

Updated by Lucas Di Pentima almost 7 years ago

Great, this LGTM.

Actions #13

Updated by Radhika Chippada almost 7 years ago

Preload collections @ a7bd5fa

Actions #14

Updated by Radhika Chippada almost 7 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:a5c53dceed8480e667249fc5fee3349bbaf20ec8.

Actions

Also available in: Atom PDF