Conversation
Support multiplexing in diann2msstats
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant ParquetReader as Parquet Reader
participant SchemaDetector as Schema Detector
participant DataProcessor as Data Processor
participant DesignParser as Design Parser
participant Merger as Table Merger
User->>ParquetReader: Request DIA-NN data
ParquetReader->>SchemaDetector: Read schema
SchemaDetector->>SchemaDetector: Check for Channel & Decoy columns
alt Multiplexing Detected (Channel exists & nunique > 1)
SchemaDetector->>DataProcessor: Multiplexed mode
DataProcessor->>DataProcessor: Expand Channel → IsotopeLabelType
DataProcessor->>DataProcessor: Filter/sanitize sequences
DataProcessor->>DesignParser: Request design with Label column
DesignParser->>DesignParser: Map SILAC labels (L/M/H)
DesignParser->>Merger: Design table with IsotopeLabelType
Merger->>Merger: Merge on [Run, IsotopeLabelType]
else Non-multiplexed Mode
SchemaDetector->>DataProcessor: Non-multiplexed mode
DataProcessor->>DataProcessor: Set default IsotopeLabelType=L
DataProcessor->>DataProcessor: Sanitize sequences
DataProcessor->>DesignParser: Request design without Label
DesignParser->>Merger: Design table
Merger->>Merger: Merge on [Run]
end
Merger->>User: Return processed MS stats
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested reviewers
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 7 |
TIP This summary will be updated as you push new changes. Give us feedback
Summary by CodeRabbit
Release Notes