-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split collections into 2 #593
base: main
Are you sure you want to change the base?
Conversation
Test Results (powered by Planemo)Test Summary
Errored Tests
Failed Tests
Passed Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
Failed Tests
Passed Tests
|
Test Results (powered by Planemo)Test Summary
Errored Tests
Failed Tests
Passed Tests
|
@wm75 if you want to review this. |
workflows/data-manipulation/split-collection/Split-collection-using-tabular-tests.yml
Outdated
Show resolved
Hide resolved
...ows/data-manipulation/split-collection/Split-collection-using-comma-separated-list-tests.yml
Outdated
Show resolved
Hide resolved
|
||
The way to split the collection differs with the workflow. | ||
|
||
- In the workflow "Split collection by pattern in identifiers", you need to specify a "pattern". This is a word that is present only in one part of your samples. This will split your collection into 2: one with the identifiers which have the 'pattern' and the other one with the identifiers which don't have. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that a regex pattern or literal words ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It uses grep so it is a regex. However, as this workflow is targeting a "user" audience I was trying to make it understandable for non bioinformaticians but I am happy to get suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They might be surpised if they do enter special characters, so best to mention this I think ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, do you expect users to run this workflow as a standalone workflow ? I thought this was useful as a thing to embed in another workflow ? Would it maybe make more sense to add what is not covered by the filter filter to the tool itself ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like I am using this workflow as a standalone. For example, to select only part of my samples for deeper analysis or for a specific heatmap etc...
You are right about the word/regex, I should warn the users.
workflows/data-manipulation/split-collection/Split-collection-by-pattern-in-identifiers.ga
Outdated
Show resolved
Hide resolved
Co-authored-by: Marius van den Beek <[email protected]>
@mvdbeek, in the workflow: "Split-collection-using-tabular", I extract during the workflow the value of the group name I want to select. Is there a way to use this value to rename the final output collections? |
See #583