Skip to content

Commit

Permalink
Remove require parent mass match smiles until bug is fixed in matchms
Browse files Browse the repository at this point in the history
  • Loading branch information
niekdejonge committed Nov 2, 2023
1 parent 442c8cb commit 0c4f014
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions library_spectra_validation/validation_pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -80,17 +80,17 @@ def __init__(self):
"require_precursor_mz": ["precursor_mz"],
"require_valid_annotation": ["smiles", "inchi", "inchikey"],
"require_correct_ionmode": ["ionmode", "adduct", "charge"],
"require_parent_mass_match_smiles": ["smiles", "parent_mass"]
# "require_parent_mass_match_smiles": ["smiles", "parent_mass"]
}
# todo require adduct, precursor mz and parent mass match.
# todo add all the checks for formatting. That everything is filled and of the expected format.
super().__init__(predefined_pipeline=None,
additional_filters=("require_precursor_mz",
"require_valid_annotation",
("require_correct_ionmode", {"ion_mode_to_keep": "both"}),
("require_parent_mass_match_smiles", {'mass_tolerance': 0.1}),
# ("require_parent_mass_match_smiles", {'mass_tolerance': 0.1}),
))

# todo add require parent mass match smiles after matchms release.
def process_spectrum(self, spectrum,
processing_report=None):
raise AttributeError("process spectrum is not a valid method of SpectrumValidator")
Expand Down

0 comments on commit 0c4f014

Please sign in to comment.