Idea #2754
closed
Optimize Workbench "run pipeline" workflow for easy use case
Description
- Separate lists of "my pipeline templates" and "shared with me". (Open "shared" tab by default if I have none of my own.)
- Javascript-searchable table similar to "show folder" page
- "Show pipeline template" page similar to "show folder" page. Instead of "contents" panel, show a list of my+shared pipeline instances that have used this template. Add "Run" button.
- On "new pipeline instance" page, default to a friendly form view optimized (for now) for selecting a single input collection.
- Any script_parameter without a "description" field does not get exposed at all
- If you have exactly one collection in your "current selection", and the pipeline requires exactly one parameter that is a collection, default to your selected collection.
- When a collection is selected, you get a preview of that collection -- at least the first few filenames, with sizes (this lets you compare with the input description which will say something like "input should have fastq or fastq.gz files")
- Click "advanced" button/tab to switch to what is now the current view, with all parameters exposed
- Option to switch to "grid" view, using cards (sample code is currently commented out on the "show folder" page)
Files
Updated by Tom Clegg over 10 years ago
- Description updated (diff)
- Story points set to 3.0
Updated by Peter Amstutz over 10 years ago
Impressions of 89096c0 (these are all subjective):
- The rounded, padded box around the instructions ("Provide a value...") looks a lot like the rounded, padded input fields, despite being a different color. Perhaps less styling would be better here?
- "Input for XXX component" seems wordy. How about just the component name with a "label" style?
- With the input field full-width, the text should be right-justified
- The instructions should go above the field, not below
- If we're going to style input field with a big, rounded, padded box, can we use the inline x-editable style instead of the popup?
- The note about the components tab at the bottom should be active, e.g. "Click on the components tab to..." instead of passive.
Updated by Tom Clegg over 10 years ago
Peter Amstutz wrote:
- The rounded, padded box around the instructions ("Provide a value...") looks a lot like the rounded, padded input fields, despite being a different color. Perhaps less styling would be better here?
Lose "alert" style, gain italics. Better, I think?
- "Input for XXX component" seems wordy. How about just the component name with a "label" style?
The word "input" is the name of the script parameter. In the case of "input" it sounds redundant. Now, if a "title" is provided in the pipeline description, use that instead of generic "{paramname} for {componentname} component".
- With the input field full-width, the text should be right-justified
Which text? I'm not picturing this...
- The instructions should go above the field, not below
I'm deferring to Bootstrap here, which suggests "bold short label above, longer description/help below". Perhaps giving the pipeline template control over both labels via "title" and "description" helps here too?
- If we're going to style input field with a big, rounded, padded box, can we use the inline x-editable style instead of the popup?
Sort of. I tried this, but even with a select box we get the "size changes when you click" bug we've run into before with inline mode. Also, you have to click twice: once to go from "static form input" to "x-editable select box" mode, then again to make the thing actually drop down. Rather than get bogged down in this, I'd like to leave it here and make our next move "pop up a full-featured select dialog that lets me search the entire system".
- The note about the components tab at the bottom should be active, e.g. "Click on the components tab to..." instead of passive.
Fixed.
Also made the form 60% instead of 100% content area width. (Better?)
→ d61f277 (and screenshot attached)
Updated by Peter Amstutz over 10 years ago
- Italics are definitely better
- Having an custom title is good. For the default title, I we should just be explicit, e.g.
"#{cname} / #{component[:script]} / #{pname.to_s.humanize}"makes totally clear the component, script and parameter that you're setting - I meant the text with the field itself, when you're not editing it, should be right justified. But since you shrunk the field down to 60% width I'm fine with it either way
- OK, Bootstrap style is valid, I think my issue was mostly with the title text
- We really ought to be able to tweak the x-editable inline editing style to not change the height of the box; however I agree that the next move for picking collections should be full-featured selection dialog
Change the default title pattern (#2) (or not, if you're really attached or have a better idea); otherwise LGTM.
Updated by Tom Clegg over 10 years ago
Peter Amstutz wrote:
- Having an custom title is good. For the default title, I we should just be explicit, e.g.
"#{cname} / #{component[:script]} / #{pname.to_s.humanize}"makes totally clear the component, script and parameter that you're setting
See attached. (I also added the words component, script, parameter because otherwise you get enigmas like "bwa-aln / bwa-aln / input".)
→ 140944b
Updated by Brett Smith over 10 years ago
Reviewing 2754-pipeline-template-description at fe84c84827af6092784408629e4cde0663fe6af8
This branch does not include the associated migration. Please add and push. The rest of the code looks good to me, but I'd like to be able to test it. :)
Updated by Brett Smith over 10 years ago
Updated by Tom Clegg over 10 years ago
- Target version changed from 2014-05-28 Pipeline Factory to 2014-06-17 Curating and Crunch