Skip to content

Commit

Permalink
fix test errors after adding parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
wolski committed Jan 17, 2025
1 parent 3b659b4 commit d822f61
Show file tree
Hide file tree
Showing 26 changed files with 34 additions and 7 deletions.
3 changes: 3 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -142,3 +142,6 @@ results.json

# secrets
**/.streamlit/secrets.toml

# wew
build.sh
2 changes: 1 addition & 1 deletion proteobench/io/params/alphapept.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ def extract_params(fname: pathlib.Path) -> ProteoBenchParameters:
params.min_precursor_charge = record["features"]["iso_charge_min"]
params.max_precursor_charge = record["features"]["iso_charge_max"]
params.enable_match_between_runs = record["workflow"]["match"] # Check if matching is enabled

params.abundance_normalization_ions = None # No normalization in AlphaPept
return params


Expand Down
2 changes: 1 addition & 1 deletion proteobench/io/params/diann.py
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,7 @@ def extract_params(fname: str) -> ProteoBenchParameters:

# If scan window is not customely set, extract it from the log file
parameters["scan_window"] = int(extract_with_regex(lines, scan_window_regex))

parameters["abundance_normalization_ions"] = None
return ProteoBenchParameters(**parameters)


Expand Down
3 changes: 3 additions & 0 deletions proteobench/io/params/fragger.py
Original file line number Diff line number Diff line change
Expand Up @@ -156,10 +156,13 @@ def extract_params(file: BytesIO) -> ProteoBenchParameters:
params.ident_fdr_protein = fragpipe_params.loc["ionquant.proteinfdr"]
params.ident_fdr_peptide = fragpipe_params.loc["ionquant.peptidefdr"]
params.ident_fdr_psm = fragpipe_params.loc["ionquant.ionfdr"]
params.abundance_normalization_ions = fragpipe_params.loc["ionquant.normalization"]

elif fragpipe_params.loc["diann.run-dia-nn"] == "true":
params.ident_fdr_protein = fragpipe_params.loc["diann.q-value"]
params.ident_fdr_peptide = fragpipe_params.loc["diann.q-value"]
params.ident_fdr_psm = fragpipe_params.loc["diann.q-value"]
params.abundance_normalization_ions = None

# Precursor charge settings
if fragpipe_params.loc["msfragger.override_charge"] == "true":
Expand Down
2 changes: 1 addition & 1 deletion proteobench/io/params/spectronaut.py
Original file line number Diff line number Diff line change
Expand Up @@ -128,7 +128,7 @@ def read_spectronaut_settings(file_path: str) -> ProteoBenchParameters:
params.second_pass = extract_value(lines, "directDIA Workflow:")
params.protein_inference = extract_value(lines, "Inference Algorithm:") # or Protein Inference Workflow:
params.predictors_library = extract_value(lines, "Hybrid (DDA + DIA) Library").replace(":", "").strip()

params.abundance_normalization_ions = extract_value(lines, "Cross-Run Normalization:")
return params


Expand Down
1 change: 1 addition & 0 deletions test/params/DIANN_1.7.16.log.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,QuantUMS high-precision
second_pass,False
protein_inference,
predictors_library,"{'RT': 'DIANN', 'IM': 'DIANN', 'MS2_int': 'DIANN'}"
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/DIANN_WU304578_report.log.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,QuantUMS high-precision
second_pass,False
protein_inference,
predictors_library,"{'RT': 'DIANN', 'IM': 'DIANN', 'MS2_int': 'DIANN'}"
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/DIANN_output_20240229_report.log.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,QuantUMS high-accuracy
second_pass,True
protein_inference,Protein_names
predictors_library,"{'RT': 'DIANN', 'IM': 'DIANN', 'MS2_int': 'DIANN'}"
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/ProlineStudio_241024.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/ProlineStudio_withMBR.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/Proline_example_2.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/Proline_example_w_Mascot_wo_proteinSets.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/Version1_9_Predicted_Library_report.log.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,QuantUMS high-precision
second_pass,True
protein_inference,Protein_names
predictors_library,"{'RT': 'DIANN', 'IM': 'DIANN', 'MS2_int': 'DIANN'}"
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/alphapept_0.4.9.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/alphapept_0.4.9_unnormalized.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/fragpipe_extracted_params.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,ProteinProphet: --maxppmdiff 2000000
predictors_library,
abundance_normalization_ions,1
1 change: 1 addition & 0 deletions test/params/fragpipe_v22_extracted_params.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,ProteinProphet: --maxppmdiff 2000000
predictors_library,
abundance_normalization_ions,1
1 change: 1 addition & 0 deletions test/params/fragpipe_win_paths_extracted_params.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,ProteinProphet: --maxppmdiff 2000000
predictors_library,
abundance_normalization_ions,1
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/i2mproteobench_params_sage_sel.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
1 change: 1 addition & 0 deletions test/params/i2mq_result_parameters_sel.csv
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,
second_pass,
protein_inference,
predictors_library,
abundance_normalization_ions,
3 changes: 2 additions & 1 deletion test/params/mqpar1.5.3.30_MBR_sel.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,6 @@
"quantification_method": null,
"second_pass": null,
"protein_inference": null,
"predictors_library": null
"predictors_library": null,
"abundance_normalization_ions": null
}
3 changes: 2 additions & 1 deletion test/params/mqpar_MQ1.6.3.3_MBR_sel.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,6 @@
"quantification_method": null,
"second_pass": null,
"protein_inference": null,
"predictors_library": null
"predictors_library": null,
"abundance_normalization_ions": null
}
3 changes: 2 additions & 1 deletion test/params/mqpar_MQ2.1.3.0_noMBR_sel.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,6 @@
"quantification_method": null,
"second_pass": null,
"protein_inference": null,
"predictors_library": null
"predictors_library": null,
"abundance_normalization_ions": null
}
3 changes: 2 additions & 1 deletion test/params/mqpar_mq2.6.2.0_1mc_MBR_sel.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,6 @@
"quantification_method": null,
"second_pass": null,
"protein_inference": null,
"predictors_library": null
"predictors_library": null,
"abundance_normalization_ions": null
}
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,4 @@ quantification_method,MS2
second_pass,directDIA+ (Deep)
protein_inference,IDPicker
predictors_library,False
abundance_normalization_ions,True

0 comments on commit d822f61

Please sign in to comment.