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

Add qc flags #106

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

Add qc flags #106

wants to merge 45 commits into from

Conversation

jdavis3141
Copy link
Contributor

  • Finalizing qc_flags functionality
  • Writes out tables and flag tables for each qc level
  • Includes the option to filter samples that fail qc flags (at normalized_counts)

@jdavis3141 jdavis3141 requested a review from yuhjong24 March 20, 2025 13:25
Copy link
Contributor

@uwidocki uwidocki left a comment

Choose a reason for hiding this comment

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

check out comment about line 113 in src/qc_table_functions.R

@@ -125,7 +142,7 @@ pipeline {
string(name: 'SAMPLE_META', defaultValue: 'sample_meta.csv', description: 'File name in BUILD_DIR of the sample meta.')
string(name: 'CELL_SET_AND_POOL_META', defaultValue: 'cell_set_and_pool_meta.csv', description: 'Cell set and pool information for this run.')
string(name: 'CELL_LINE_META', defaultValue: 'cell_line_meta.csv', description: 'File in BUILD_DIR containing cell line metadata')
string(name: 'CONTROL_BARCODE_META', defaultValue: 'h-b', description: 'Metadata for control barcodes. If the CBs exist in cellDB, this can simply be the cb_ladder name (ie, h-a). Otherwise, this must be a csv file located in the build directory.')
string(name: 'CONTROL_BARCODE_META', defaultValue: 'h-a', description: 'Metadata for control barcodes. If the CBs exist in cellDB, this can simply be the cb_ladder name (ie, h-a). Otherwise, this must be a csv file located in the build directory.')
Copy link
Contributor

Choose a reason for hiding this comment

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

should we specify that the string needs to be all lowercase, too since some people write it as "H-a"?

jdavis3141 and others added 3 commits March 20, 2025 11:59
Added extra condition so that contaminants are not counted as recovered cell lines
… if it exists to prevent issues with re-runs
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