Skip to content
Snippets Groups Projects

Metadata output update

Merged Gervaise Henry requested to merge new.qc.out into develop

Cumulative update focused on metadata output:

  • Add read length: both submitter entered and inferred (median length)
  • Add calculated read counts: both raw and assigned
  • Add median TIN
  • Output multiqc json (for future use)
  • Convert gene symbols to Entrez ID: for data-hub convenience (use version specific metadata - added to references)
  • Create bash script to initiate parallel replicate runs from study RID
  • Change consistence tests for assigned reads exact match to being within +/- 1/100,000 of expected

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • Documentation in docs is updated
  • Replace dag.png with the most recent CI pipleine integrated_pe artifact
  • CHANGELOG.md is updated
  • README.md is updated
  • LICENSE.md is updated with new contributors
  • Docker images moved to production release and changed in pipeline
  • Docker images used in the CI unit tests match those used in pipeline

/cc @ghenry @venkat.malladi

Edited by Gervaise Henry

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Author Owner

      Run in repo root dir: sh workflow/scripts/splitStudy.sh [studyRID]

      It will run in parallel in batches of 5 replicatesRID (this can be changed in splitStudy.sh: xargs -P flag)

    • Please register or sign in to reply
  • Gervaise Henry marked the checklist item This comment contains a description of changes (with reason) as completed

    marked the checklist item This comment contains a description of changes (with reason) as completed

  • Gervaise Henry marked the checklist item If you've fixed a bug or added code that should be tested, add tests! as completed

    marked the checklist item If you've fixed a bug or added code that should be tested, add tests! as completed

  • Gervaise Henry marked the checklist item Documentation in docs is updated as completed

    marked the checklist item Documentation in docs is updated as completed

  • Gervaise Henry marked the checklist item Documentation in docs is updated as incomplete

    marked the checklist item Documentation in docs is updated as incomplete

  • Gervaise Henry marked the checklist item README.md is updated as completed

    marked the checklist item README.md is updated as completed

  • Gervaise Henry marked the checklist item *Close issue* as completed

    marked the checklist item *Close issue* as completed

  • Gervaise Henry unmarked as a Work In Progress

    unmarked as a Work In Progress

  • Gervaise Henry added 2 commits

    added 2 commits

    • 506a006b - Update dag
    • ade9acec - Change consistence test from exact match to within 1/100,000 change from expected

    Compare with previous version

  • Gervaise Henry changed the description

    changed the description

  • Gervaise Henry marked the checklist item Docker images used in the CI unit tests match those used in pipeline as completed

    marked the checklist item Docker images used in the CI unit tests match those used in pipeline as completed

  • Gervaise Henry marked the checklist item Replace dag.png with the most recent CI pipleine integrated_pe artifact as completed

    marked the checklist item Replace dag.png with the most recent CI pipleine integrated_pe artifact as completed

  • Gervaise Henry added 1 commit

    added 1 commit

    • 6341ca53 - Convert string to int for range checking in consistency test

    Compare with previous version

  • merged

  • Venkat Malladi mentioned in commit fe51a743

    mentioned in commit fe51a743

Please register or sign in to reply