Skip to content

Conversation

@ldgauthier
Copy link
Contributor

Wrong file name in WGS joint genotyping intervals prevents scatters wider than ~210

Checklist

If you can answer "yes" to the following items, please add a checkmark next to the appropriate checklist item(s) and notify our WARP documentation team by tagging either @ekiernan or @kayleemathews in a comment on this PR.

  • Did you add inputs, outputs, or tasks to a workflow?
  • Did you modify, delete or move: file paths, file names, input names, output names, or task names?
  • If you made a changelog update, did you update the pipeline version number?

@ekiernan

Wrong file name in WGS intervals prevents scatters wider than ~210
@dsde-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

1 similar comment
@dsde-jenkins
Copy link
Collaborator

Can one of the admins verify this patch?

@github-actions
Copy link

Remember to squash merge!

@jessicaway
Copy link
Member

add to whitelist

@github-actions
Copy link

Remember to squash merge!

@github-actions
Copy link

🔍Changelog Validation Results:

basename: missing operand
Try 'basename --help' for more information.
Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link

🔍Version Validation Results:

basename: missing operand
Try 'basename --help' for more information.
Comparing versions and changelogs for pipelines that differ from the versions on 'origin/staging':
All WDLs and changelog files appear to be valid for this release.

@nikellepetrillo
Copy link
Contributor

@ldgauthier @rickymagner Do you still want this PR merged in? Is this the interval list that should be used in our WARP tests?

@ldgauthier
Copy link
Contributor Author

Yes please! I believe it's the same territory, but split wider, so tests shouldn't change, but it would like to use this for tests as well.

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.

4 participants