Test pull request

[mbiletska]

Hi,
My name is Maryna, and I created an SDRF annotation for PXD009172.
The file was validated using the SDRF Validator and manually reviewed against the associated publication.
Thanks!

[qodo-code-review]

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)

• .DS_Store is excluded by !**/.DS_Store
• datasets/PXD009172/PXD009172.sdrf.tsv is excluded by !**/*.tsv

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: bf79e779-fadf-4f69-9c16-7c06374636fe

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Pull request overview

Adds a new SDRF annotation file for ProteomeXchange dataset PXD009172 under the repository’s datasets/ collection, expanding the set of curated, validator-ready SDRF TSVs for downstream parsing/validation workflows.

Changes:

• Added datasets/PXD009172/PXD009172.sdrf.tsv containing sample- and run-level metadata for PXD009172.
• Populated key MS acquisition, instrument, digestion, tolerance, and modification metadata columns for 21 assays.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[ypriverol]

Thanks for the PXD009172 annotation, @mbiletska! The SDRF passes parse_sdrf validate-sdrf structurally, the 21 sample rows map cleanly to the actual .raw files in the PRIDE record, and the search parameters (Trypsin, fixed Carbamidomethyl, variable Oxidation, 4.5 ppm precursor / 0.5 Da fragment) match the publication's methods. Nice work anchoring it to the source.

A few issues to address before merge (details inline):

1. .DS_Store committed — a macOS artifact was added at the repo root; please remove it and consider adding it to .gitignore.
2. Instrument — listed as Orbitrap Velos Pro, but the PRIDE record and the paper (PMID: 30510230) say LTQ Orbitrap Velos (a different PSI-MS instrument). Recommend NT=LTQ Orbitrap Velos;AC=MS:1001742, which also matches the NT=...;AC=... format used by every other file in the repo.
3. Missing comment[sdrf version] / comment[sdrf template] — present in 307/308 existing files; please add them for consistency.
4. Minor — the two olfactory-disorder disease labels trigger ontology warnings (not errors); see the inline note.

Items 1–3 are worth fixing; 4 is optional. Happy to help once updated.

---

Generated by Claude Code

[ypriverol]

Missing comment[sdrf version] and comment[sdrf template] columns.

307 of the 308 existing dataset files in this repo include these two trailing columns — this PR would be the only file without them. Please add them, e.g.:

• comment[sdrf version] → 1.1
• comment[sdrf template] → default (or human, matching the template you annotated against)

Minor (disease ontology): parse_sdrf validate-sdrf passes, but emits warnings that idiopathic olfactory disorder and post-infectious olfactory disorder in characteristics[disease]/factor value[disease] aren't found in the disease ontologies (mondo;efo;doid;ncit;pato). If a matching ontology term exists, prefer the NT=...;AC=... form; otherwise it's fine to keep the study-defined labels — just flagging since CI surfaces them.

---

Generated by Claude Code

[ypriverol]

@nithujohn Can you help here?