Skip to content

feat: added patched path and spectrumreader - #60

Merged
sander-willems-bruker merged 3 commits into
mainfrom
feature/patched_spectrum_reader
Aug 18, 2026
Merged

feat: added patched path and spectrumreader#60
sander-willems-bruker merged 3 commits into
mainfrom
feature/patched_spectrum_reader

Conversation

@sander-willems-bruker

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR adds optional (patched feature) support for a new “patched” timsTOF path type and wires it through the spectrum/precursor readers and converter selection so the crate can recognize and operate on patched datasets.

Changes:

  • Add TimsTofFileType::Patched(...) and detection in TimsTofPath::new.
  • Integrate timsrust_patched::PatchedSpectrumReader into SpectrumReader (with random-access explicitly unsupported).
  • Handle the patched file type in precursor reader construction and in m/z, IM, and RT converter selection.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
crates/timsrust/src/timstof.rs Adds a patched file type variant and path auto-detection logic.
crates/timsrust/src/spectrum_reader.rs Adds a patched-backed spectrum reader variant plus an error for unsupported random access.
crates/timsrust/src/precursor_reader.rs Rejects patched datasets for precursor reading and introduces an error variant.
crates/timsrust/src/converters.rs Adds patched file type handling for converter construction.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread crates/timsrust/src/timstof.rs
Comment thread crates/timsrust/src/precursor_reader.rs
Comment thread crates/timsrust/src/spectrum_reader.rs
Comment thread crates/timsrust/src/spectrum_reader.rs
sander-willems-bruker and others added 2 commits August 18, 2026 13:37
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@sander-willems-bruker
sander-willems-bruker merged commit eb841e6 into main Aug 18, 2026
1 check passed
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.

3 participants