Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated metabo-batches example #233

Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

Conversation

mwalzer
Copy link
Collaborator

@mwalzer mwalzer commented Jul 23, 2024

No description provided.

@mwalzer mwalzer mentioned this pull request Jul 23, 2024
7 tasks
This was linked to issues Jul 25, 2024
specification_documents/examples/metabo-batches.mzQC Outdated Show resolved Hide resolved
docs/pages/worked-examples/metabo-batches.mzQC.md Outdated Show resolved Hide resolved
docs/pages/worked-examples/metabo-batches.mzQC.md Outdated Show resolved Hide resolved
@bittremieux bittremieux self-requested a review October 18, 2024 11:18
bittremieux and others added 23 commits January 10, 2025 12:01
* Update set mzQC for validator

* Update set description

* Add temporary accession number

* Update example

* Fix typo

Co-authored-by: Chris Bielow <[email protected]>

* Fix protein contaminant metric

* Update the mzQC file as well

* Update the OBO version

* Updated CV term definition

---------

Co-authored-by: Chris Bielow <[email protected]>
* Update set mzQC for validator

* Update set description

* Add temporary accession number

* Update example

* Update mzML example

* Add clarification about table metrics

* Update set mzQC for validator

* Update set description

* Add temporary accession number

* Update example

* Update mzML example

* Add clarification about table metrics
* updated USI example to mzQC v1.0

* Update USI-example.mzQC

fix validation issues

* Some editing

---------

Co-authored-by: Mathias Walzer <[email protected]>
Co-authored-by: Wout Bittremieux <[email protected]>
* updated QC2 sample examples,
file validation fixes,
explanation corrections

* QC2 sample example markdown formatting

* Update example

* Update file names

* Spelling

* Fix duplicate file names

* Update CV version

---------

Co-authored-by: Wout Bittremieux <[email protected]>
* Linkcheck on PR

fixes  #241

* Avoid redundant conditions

---------

Co-authored-by: Wout Bittremieux <[email protected]>
Copy link
Collaborator

@bittremieux bittremieux left a comment

Choose a reason for hiding this comment

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

Ok, I messed up the git stuff a bit somehow. I don't think anything was lost, so should be fine.

@bittremieux
Copy link
Collaborator

One minor question. The metric in the individual runQualities is "number of identified quantification data points", whereas the PCA metric is "identified MS1 feature area principal component analysis result". So the number of data points vs feature area definitions don't fully seem to line up here?

@bittremieux
Copy link
Collaborator

Issues reported by the validator:

  • Metadata labels of severity 6 and message: Run/SetQuality label is not unique in file!
  • ID based metric but no ID input file of severity 6 and message: ID based metrics present but no ID input file could be found registered in the mzQC file: run/set label =

@bittremieux
Copy link
Collaborator

@mwalzer to look into resolving the discrepancy between run-level and set-level metrics.

@mwalzer
Copy link
Collaborator Author

mwalzer commented Jan 10, 2025

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.

Update examples to mzQC v1.0 Remove QC CV from markup docs
4 participants