Skip to content

Add acquisition_software_name/version fields once openmassspec-core hits 1.4.0 #31

Description

@Nathan-D-R

openmassspec-core #16 will bump 1.3.0 -> 1.4.0, adding two new required RunMetadata fields (acquisition_software_name, acquisition_software_version, both Option<String>, no Default impl). This repo's production RunMetadata literal at crates/opentimstdf/src/mzml.rs:558 (run_metadata_for) will fail to compile once the dependency is bumped, until those two fields are added (plus two more test-only construction sites at mzml.rs:658/672).

Minimal fix: acquisition_software_name: None, acquisition_software_version: None,. Worth checking first whether the TDF SQLite schema's metadata table exposes a timsControl/Compass version - if so, wiring up a real value is a small bonus on top of the minimal fix.

Blocked on OpenMassSpecCore#16 shipping first.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions