Skip to content

Create README.md for QTL analysis#1729

Merged
ekiernan merged 12 commits intodevelopfrom
lk_aou_add_rna_documentation
Jan 30, 2026
Merged

Create README.md for QTL analysis#1729
ekiernan merged 12 commits intodevelopfrom
lk_aou_add_rna_documentation

Conversation

@ekiernan
Copy link
Copy Markdown
Contributor

Adding a first draft readme for RNA qtl analysis for AoU v9

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

@github-actions
Copy link
Copy Markdown

🔍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.

Copy link
Copy Markdown
Contributor

@nikellepetrillo nikellepetrillo left a comment

Choose a reason for hiding this comment

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

looks good!

Copy link
Copy Markdown
Contributor

@khajoue2 khajoue2 left a comment

Choose a reason for hiding this comment

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

lgtm

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 9, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 9, 2026

🔍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
Copy Markdown

github-actions Bot commented Jan 9, 2026

🔍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.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 9, 2026

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 9, 2026

🔍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
Copy Markdown

github-actions Bot commented Jan 9, 2026

🔍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.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍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.

@github-actions
Copy link
Copy Markdown

🔍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
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍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
Copy Markdown

🔍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.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@ekiernan ekiernan requested a review from Copilot January 29, 2026 19:22
@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds comprehensive documentation for the All of Us RNA-seq QTL analysis pipeline and enhances error handling across multiple WDL workflows by adding bash strict mode (set -euo pipefail).

Changes:

  • Created a detailed README documenting the end-to-end eQTL and sQTL analysis workflow
  • Added set -euo pipefail to command blocks in multiple WDL tasks for improved error handling
  • Incremented pipeline versions and updated changelogs to reflect these improvements

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
all_of_us/rna_seq/README.md New comprehensive documentation covering the complete QTL analysis pipeline workflow
all_of_us/rna_seq/susieR_workflow.wdl Added bash strict mode to susieR task
all_of_us/rna_seq/susieR_workflow.changelog.md Version bump to aou_9.0.1 with changelog entry
all_of_us/rna_seq/aggregate_rsem_results.wdl Added bash strict mode and version bump to aou_9.0.1
all_of_us/rna_seq/aggregate_rsem_results.changelog.md Added changelog entry for version aou_9.0.1
all_of_us/rna_seq/CalculatePhenotypeGroups.wdl Added bash strict mode to PrepareSpliceData task
all_of_us/rna_seq/CalculatePhenotypeGroups.changelog.md Added changelog entry for version aou_9.0.1
all_of_us/rna_seq/AggregateSusieWorkflow.wdl Added bash strict mode to two tasks and version bump to aou_9.0.2
all_of_us/rna_seq/AggregateSusieWorkflow.changelog.md Added changelog entry for version aou_9.0.2

Comment thread all_of_us/rna_seq/README.md Outdated
Comment thread all_of_us/rna_seq/README.md Outdated
Comment thread all_of_us/rna_seq/README.md Outdated
# aou_9.0.1
2026-01-29 (Date of Last Commit)

* Added set euo pipefail to tasks
Copy link

Copilot AI Jan 29, 2026

Choose a reason for hiding this comment

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

Missing hyphen in bash option. Should be 'set -euo pipefail' to correctly describe the bash command.

Suggested change
* Added set euo pipefail to tasks
* Added `set -euo pipefail` to tasks

Copilot uses AI. Check for mistakes.
Comment thread all_of_us/rna_seq/aggregate_rsem_results.changelog.md
Comment thread all_of_us/rna_seq/susieR_workflow.changelog.md
Comment thread all_of_us/rna_seq/AggregateSusieWorkflow.changelog.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Remember to squash merge!

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

Remember to squash merge!

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

1 similar comment
@github-actions
Copy link
Copy Markdown

🔍Changelog Validation Results:

Comparing changelogs for pipelines that differ from the versions on 'origin/develop':
All changelog files are valid for this release.

@github-actions
Copy link
Copy Markdown

🔍Version Validation Results:

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.

@ekiernan ekiernan merged commit 59e915a into develop Jan 30, 2026
10 checks passed
@ekiernan ekiernan deleted the lk_aou_add_rna_documentation branch January 30, 2026 17:03
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