Skip to content

Conversation

@vagisha
Copy link
Collaborator

@vagisha vagisha commented Apr 16, 2025

Rationale

  • DIA-NN 2.0 produces a Parquet report file instead of a TSV
  • File extensions for raw files are no longer included in the BLIB since they are missing from the DIA-NN results

Related Pull Requests

Changes

  • Look for a Parquet file first. If not found, then look for a TSV report file.
  • Allow base name matching for raw files since file extensions may be missing in the SpectrumSourceFiles table of the BLIB
  • Updated test to validate that both file types are correctly detected and that raw file matching works even in the absence of file extensions.

vagisha added 3 commits April 14, 2025 18:16
- Report file is a Parquet file
- Since file extensions are not included in the raw file names, allow base name matching with valid raw files
- Updated test, and added test files
@vagisha vagisha requested a review from labkey-jeckels April 16, 2025 02:55
@vagisha vagisha merged commit 3745456 into release25.3-SNAPSHOT Apr 20, 2025
3 checks passed
@vagisha vagisha deleted the 25.3_fb_diann-2.0-libraries branch April 20, 2025 20:44
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