-
Notifications
You must be signed in to change notification settings - Fork 13
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'main' into ci_test_validate
- Loading branch information
Showing
15 changed files
with
224 additions
and
28 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,15 +7,13 @@ jobs: | |
runs-on: ubuntu-latest | ||
name: Check for broken links | ||
steps: | ||
- name: Check for broken links | ||
id: link-report | ||
uses: celinekurpershoek/[email protected] | ||
- name: Setup Python | ||
uses: actions/setup-python@v5 | ||
with: | ||
# Required: | ||
url: 'https://hupo-psi.github.io/mzQC/' | ||
# optional: | ||
honorRobotExclusions: false | ||
ignorePatterns: 'github,google' | ||
recursiveLinks: false # Check all URLs on all reachable pages (could take a while) | ||
python-version: '3.10' | ||
- run: pip3 install linkchecker | ||
- name: Link Checker | ||
id: link-report | ||
run: linkchecker -ofailures --check-extern --ignore-url="\s*\.md" https://hupo-psi.github.io/mzQC/ | ||
- name: Get the result | ||
run: echo "${{steps.link-report.outputs.result}}" | ||
run: echo linkchecker/failures |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
# HUPO-PSI Spring Meeting 2024 | ||
|
||
## March 19, 2024 | ||
|
||
- Wout Bittremieux | ||
- Kozo Nishida | ||
- Tim Van Den Bossche | ||
- Mathias Walzer | ||
|
||
## Tutorials and examples | ||
|
||
Going through the [tutorials and examples](https://hupo-psi.github.io/mzQC/examples/) to make sure they are compliant with the 1.0.0 format. Some tutorials were fully updated, some need new CV terms, others still need to be revised. | ||
|
||
- Common issues with the examples: | ||
- CV version needs to be updated. | ||
- Some outdated CV terms used. | ||
- Units were not always specified for all relevant metrics. | ||
- Make sure that the file name and location match, and that the latter is a proper URI. | ||
- Warnings by the validator if optional information is missing. For completeness of these examples, optional information has now been included. | ||
- Reorder examples going from introduction to mid-level to advanced topics. | ||
- Issues in the Python notebooks could be resolved by upgrading the Python version to 3.10+. Some links fixes needed as well. | ||
|
||
This also revealed some bugs in pymzqc: | ||
|
||
- Installation issues due to outdated version of Pronto and fastobo. | ||
- Activate GitHub Actions for Python 3.10+. | ||
- Incorrect specification of `unit` in `qualityMetric`, make it of ctype `cvParameter`. | ||
- Allow `proteinGroups.txt` from MaxQuant as an identification file. | ||
- Fix installation of offline validator script. | ||
|
||
Validation issues: | ||
|
||
- Faulty URIs flagged as inconsistent inputs. | ||
- The file extension is inconsistently split from the input file name and location. Instead don't remove the extension prior to checking whether these two correspond to each other. | ||
- Display the active version number on the validator web page. | ||
|
||
## Promoting mzQC | ||
|
||
Most current material is geared towards bioinformaticians. Instead, our main audience should be the broader scientific community, to educate them on how QC can improve their work. | ||
Therefore, besides technical tutorials and examples, we should clearly describe which end-to-end tools are available and how to harness mzQC. This already needs to be obvious from the website's main page! | ||
Some more details were added to the [tools page](https://hupo-psi.github.io/mzQC/resource-guide/#qc-software), but this still needs to be significantly extended with information on the various tools implementing mzQC and which QC metrics they can calculate. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
# QC working group teleconference 3 April 2024 | ||
|
||
- Chris Bielow | ||
- Wout Bittremieux | ||
- Tim Van Den Bossche | ||
- Mathias Walzer | ||
|
||
--- | ||
|
||
## mzQC software libraries manuscript | ||
|
||
Rather than synchronizing publication of the manuscript with the main manuscript describing the mzQC format itself, we decided to submit it to the [JASMS special issue on computational mass spectrometry](https://axial.acs.org/analytical-chemistry/jasms-call-for-papers-special-issue-on-computational-mass-spectrometry) (submission deadline April 30, 2024). | ||
|
||
Technical aspects related to finalizing the manuscript: | ||
- Move away from the Nextflow analysis workflow because it just added complexity and compatibility issue. It is also overkill for this limited example. | ||
- Discussion on the necessity of proper FDR control, including lowering the FDR threshold to 1% to account for E. coli's small proteome, and the need for protein-level FDR control, as we report protein results. | ||
- All software libraries and the analysis repository on GitHub need to be brought up to date matching the results in the manuscript. Repository cleanup can still happen after the manuscript submission as well. | ||
- Discussion on how to represent the results of the QC analysis in the manuscript. Tim had some suggestions and will try to clarify the figure, after Mathias shares the latest updated version. | ||
- Requests for missing CV terms were submitted to the PSI-MS-CV repository. | ||
|
||
Related discussions: | ||
- mzQC file merging is a relevant task that should be easily supported by the software libraries. For the current manuscript a somewhat hacked-together solution initially suffices. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
# QC working group teleconference 17 April 2024 | ||
|
||
- Chris Bielow | ||
- Wout Bittremieux | ||
- Nils Hoffmann | ||
- Dave Tabb | ||
- Tim Van Den Bossche | ||
- Mathias Walzer | ||
|
||
--- | ||
|
||
## Working group charter | ||
|
||
We went over the [working group charter](https://www.psidev.info/quality-control-working-group-charter) with an eye on its annual update. Some minor updates (affiliation update), but other than that, no major changes are needed. | ||
|
||
Our key effort during the next year will be to demonstrate the benefits and applications of mzQC in practice, through the development of use cases and support of QC software. | ||
|
||
Wout will update the charter and send it for consultation to Slack. | ||
|
||
## CV | ||
|
||
- Precursor mass deviation ([#254](https://github.com/HUPO-PSI/psi-ms-CV/pull/254)): Good to be merged. Note that FragPipe reports the absolute mass deviation (although not explicitly as a QC metric). | ||
- Number of missed cleavages ([#255](https://github.com/HUPO-PSI/psi-ms-CV/pull/255)): Discussion on how to represent this metric and its definition. Mathias will apply some refinements. | ||
- identified MS2 vs RT ([#255](https://github.com/HUPO-PSI/psi-ms-CV/pull/255)): Some confusion around what this metric precisely represents, indicating that its definition should be clarified. Mathias will update it in analogy to some existing related terms. | ||
|
||
## mzQC software libraries manuscript | ||
|
||
- Mathias reported on the updated data analysis workflow (including protein-level FDR control, which necessitated switching search engines). | ||
- Mathias will share the updated data analysis figure with Tim for some visual edits. | ||
- The [JASMS special issue on computational mass spectrometry](https://axial.acs.org/analytical-chemistry/jasms-call-for-papers-special-issue-on-computational-mass-spectrometry) submission deadline is April 30, 2024. We will have an additional meeting next week to finalize the manuscript by this deadline. |
Oops, something went wrong.