Skip to content

Conversation

qclayssen
Copy link
Collaborator

@qclayssen qclayssen commented Aug 21, 2025

Request from Sehrish slack

@qclayssen qclayssen self-assigned this Aug 26, 2025
@qclayssen qclayssen marked this pull request as ready for review August 28, 2025 00:53
@qclayssen qclayssen added the enhancement New feature or request label Aug 28, 2025
@qclayssen qclayssen requested a review from scwatts August 28, 2025 00:53




Copy link
Member

Choose a reason for hiding this comment

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

Considering reverting if for nothing other than consistency

cat <<-END_VERSIONS > versions.yml
"${task.process}":
vcf2maf: 1.6.22
Copy link
Member

Choose a reason for hiding this comment

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

Let's switch this to the standard 'no version' template

tag "${meta.id}"
label 'process_medium'

container 'docker.io/qclayssen/vcf2maf:debian_v1.6.22'
Copy link
Member

Choose a reason for hiding this comment

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

To be pushed the UMCCR account as you mentioned

}

genome {
build = 'GRCh38'
Copy link
Member

Choose a reason for hiding this comment

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

The genome block is reserved for filepaths and since we'll only use GRCh38 with sash in the foreseeable future, I would remove this and hardcode the value into the corresponding argument in the VCF2MAF process

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants