Skip to content

Commit 09aae6f

Browse files
committed
Corrected tests after adding more details in qcData sequencing_metrics
1 parent c9bb466 commit 09aae6f

File tree

6 files changed

+44
-11
lines changed

6 files changed

+44
-11
lines changed

checkQC/parsers/illumina.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ def from_bclconvert(cls, runfolder_path, parser_config):
3636
sequencing_metrics = {
3737
lane + 1: {
3838
"total_cluster_pf": summary.at(0).at(lane).reads_pf(),
39-
"total_cluster": summary.at(0).at(lane).reads(),
4039
"pf_clusters": summary.at(0).at(lane).cluster_count_pf().mean(),
4140
"raw_clusters": summary.at(0).at(lane).cluster_count().mean(),
4241
"raw_density":summary.at(0).at(lane).density().mean(),

requirements/prod

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
click~=8.1.1
22
PyYAML~=6.0
3-
interop~=1.3.2
3+
interop~=1.4.0
44
xmltodict~=0.13.0
55
tornado~=6.3.2
66
sample_sheet~=0.13.0
77
pandas~=2.2.2
8-
numpy~=1.26.4
8+
numpy~=2.2.4
99
samshee~=0.2.3
1010
jsonschema~=4.23.0

setup.py

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,19 @@
1111
author_email='[email protected]',
1212
url="https://www.github.com/Molmed/checkQC",
1313
download_url='https://github.com/Molmed/checkQC/archive/{}.tar.gz'.format(__version__),
14-
python_requires='>3.10, <3.11',
14+
python_requires='>3.10',
1515
install_requires=[
16-
"click",
17-
"PyYAML>=6.0",
18-
"interop>=1.2.4",
19-
"xmltodict",
20-
"tornado",
21-
"sample_sheet"],
16+
"click~=8.1.1",
17+
"PyYAML~=6.0",
18+
"interop~=1.4.0",
19+
" xmltodict~=0.13.0",
20+
"tornado~=6.3.2",
21+
" sample_sheet~=0.13.0",
22+
" pandas~=2.2.2",
23+
"numpy~=2.2.4",
24+
"samshee~=0.2.3",
25+
"jsonschema~=4.23.0",
26+
],
2227
packages=find_packages(exclude=["tests*"]),
2328
test_suite="tests",
2429
package_data={

tests/parsers/test_illumina_parser.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -107,4 +107,5 @@ def test_read_samplesheet(runfolder_path):
107107
'lane': 1,
108108
'sample_id': 'Sample_14574-Qiagen-IndexSet1-SP-Lane1',
109109
'sample_project': 'AB-1234',
110+
'description': 'LIBRARY_NAME:test',
110111
}

tests/resources/bclconvert/200624_A00834_0183_BHMTFYTINY/SampleSheet.csv

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ FastqCompressionFormat,gzip,,
1818
SoftwareVersion,4.1.5,,
1919
,,,
2020
[BCLConvert_Data],,,
21-
Lane,Sample_ID,Index,Index2,Sample_Project, Description
21+
Lane,Sample_ID,Index,Index2,Sample_Project,Description
2222
1,Sample_14574-Qiagen-IndexSet1-SP-Lane1,GAACTGAGCG,TCGTGGAGCG,AB-1234,LIBRARY_NAME:test
2323
1,Sample_14575-Qiagen-IndexSet1-SP-Lane1,AGGTCAGATA,CTACAAGATA,CD-5678,LIBRARY_NAME:test
2424
2,Sample_14574-Qiagen-IndexSet1-SP-Lane2,GAACTGAGCG,TCGTGGAGCG,AB-1234,LIBRARY_NAME:test

tests/test_qc_data.py

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,33 +35,41 @@ def bclconvert_runfolder():
3535
"index": "GAACTGAGCG",
3636
"index2": "TCGTGGAGCG",
3737
"sample_project": "AB-1234",
38+
"description": "LIBRARY_NAME:test",
3839
},
3940
{
4041
"lane": 1,
4142
"sample_id": "Sample_14575-Qiagen-IndexSet1-SP-Lane1",
4243
"index": "AGGTCAGATA",
4344
"index2": "CTACAAGATA",
4445
"sample_project": "CD-5678",
46+
"description": "LIBRARY_NAME:test",
4547
},
4648
{
4749
"lane": 2,
4850
"sample_id": "Sample_14574-Qiagen-IndexSet1-SP-Lane2",
4951
"index": "GAACTGAGCG",
5052
"index2": "TCGTGGAGCG",
5153
"sample_project": "AB-1234",
54+
"description": "LIBRARY_NAME:test",
5255
},
5356
{
5457
"lane": 2,
5558
"sample_id": "Sample_14575-Qiagen-IndexSet1-SP-Lane2",
5659
"index": "AGGTCAGATA",
5760
"index2": "CTACAAGATA",
5861
"sample_project": "CD-5678",
62+
"description": "LIBRARY_NAME:test",
5963
},
6064
],
6165
},
6266
"expected_sequencing_metrics": {
6367
1: {
6468
"total_cluster_pf": 532_464_327,
69+
"pf_clusters": 3_413_232.5,
70+
"raw_clusters": 4_091_904.0,
71+
"raw_density": 2_961_270.5,
72+
"pf_density": 2_470_118.25,
6573
"yield": 122_605_416,
6674
"yield_undetermined": 121_940_136,
6775
"top_unknown_barcodes": {
@@ -113,15 +121,27 @@ def bclconvert_runfolder():
113121
{
114122
"sample_id": "Sample_14574-Qiagen-IndexSet1-SP-Lane1",
115123
"cluster_count": 9920,
124+
"percent_of_lane": 0.29,
125+
"percent_perfect_index_reads": 97.96,
126+
"mean_q30": 36.37,
127+
"percent_q30": 0.96,
116128
},
117129
{
118130
"sample_id": "Sample_14575-Qiagen-IndexSet1-SP-Lane1",
119131
"cluster_count": 8560,
132+
"percent_of_lane": 0.25,
133+
"percent_perfect_index_reads": 98.15,
134+
"mean_q30": 36.43,
135+
"percent_q30": 0.96,
120136
},
121137
],
122138
},
123139
2: {
124140
"total_cluster_pf": 530_917_565,
141+
"pf_clusters": 3_403_318.25,
142+
"raw_clusters": 4_091_904.0,
143+
"raw_density": 2_961_270.5,
144+
"pf_density": 2_462_942.5,
125145
"yield": 124_497_108,
126146
"yield_undetermined": 123_817_428,
127147
"top_unknown_barcodes": {
@@ -173,10 +193,18 @@ def bclconvert_runfolder():
173193
{
174194
"sample_id": "Sample_14574-Qiagen-IndexSet1-SP-Lane2",
175195
"cluster_count": 10208,
196+
"percent_of_lane": 0.3,
197+
"percent_perfect_index_reads": 98.2,
198+
"mean_q30": 36.4,
199+
"percent_q30": 0.96,
176200
},
177201
{
178202
"sample_id": "Sample_14575-Qiagen-IndexSet1-SP-Lane2",
179203
"cluster_count": 8672,
204+
"percent_of_lane": 0.25,
205+
"percent_perfect_index_reads": 98.29,
206+
"mean_q30": 36.48,
207+
"percent_q30": 0.97,
180208
},
181209
],
182210
},

0 commit comments

Comments
 (0)