Skip to content
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

Use id column from samplesheet if specified #8

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

adamjtaylor
Copy link
Collaborator

For HTAN we stage the outputs from nf-artist using the original synapse ID of the file as part of the URL, for example:

https://d3p249wtgzkn5u.cloudfront.net/synid/syn25075713/stoic_franklin/thumbnail.png

To allow for nf-artist to by not dependent on Synapse the current samplesheet takes a column image that is the path to a staged image.

It would be advantageous for HTAN and other projects to allow for a id column in the samplesheet that gets taken into the meta map as id if specified, if not specified the current behavior of using the image simpleName is used.

Copy link

@thomasyu888 thomasyu888 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pre-approving, but just a question.

@@ -9,7 +9,12 @@ workflow SAMPLESHEET_SPLIT {
.map {
row ->
def meta = [:]
meta.id = file(row.image).simpleName
if (row.id ) {
meta.id = row.id

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this id a synapse id?

Copy link
Collaborator Author

@adamjtaylor adamjtaylor Aug 17, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For HTAN we would use a Synapse ID, but this could be used any time you don't want the output directories to be named by the filename and want to provide a custom ID in their place

Copy link

@rxu17 rxu17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@adamjtaylor adamjtaylor merged commit e096616 into main Aug 17, 2023
3 checks passed
@adamjtaylor adamjtaylor deleted the custom_id_in_samplesheet branch August 17, 2023 22:24
@adamjtaylor adamjtaylor restored the custom_id_in_samplesheet branch August 24, 2023 13:26
@adamjtaylor adamjtaylor deleted the custom_id_in_samplesheet branch August 24, 2023 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants