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

OpenMS 3.1 #697

Merged
merged 53 commits into from
Jun 14, 2024
Merged

OpenMS 3.1 #697

merged 53 commits into from
Jun 14, 2024

Conversation

bernt-matthias
Copy link
Collaborator

@bernt-matthias bernt-matthias commented Dec 22, 2022

For preparation .. collecting fixes :)

  • truncate description to the first line: MSFragger contained a huge disclaimer :(
  • also GNPSExport has a long description
  • Fix / improve XMLValidator add mzml-validation tool RECETOX/galaxytools#319
  • try to use remote test data as much as possible
    • is it possible to rely on data from OpenMS git as much as possible?

3.1

  • MapAlignerPoseClustering: out and trafo_out are not OPTIONAL_OUTPUTS .. one needs to be given.
  • Sirius deprecate all/mostly or support email/password login, but OpenMS likely will drop support anyway SiriusAdapter fails in OpenMS 3.0 OpenMS/OpenMS#7000 (comment)
  • drop adapters that require 3rd party software not available in conda
    • drop NovorAdapter (3rd party software seem not available anymore?)
    • MSFragger, MaRaClusterAdapter,MascotAdapter?
  • add expect_num_outputs
  • Fix FLASHDecov
    • replace command section parts for -out_spec FLASHDeconv_1_spec1.csv FLASHDeconv_1_spec2.csv FLASHDeconv_1_spec3.csv (and analogously for

MSFragger contained a huge disclaimer :(
full message is:

```
Note: This message can also be triggered if you run out of space in your tmp directory or (32bit OMSSA only) OMSSA ran out of RAM because chunking was not used (that's the default) or 'chunk_size' was too large (>30k). Look above!
```
full message in stderr is:

```
Process '/gpfs1/data/galaxy_server/galaxy/database/dependencies/_conda/envs/mulled-v1-4ada2d9266641ecd767c7de738261cde57c65095a0fa06c286e3bc2f08b0cd57/bin/comet' crashed hard (segfault-like). Please check the log.
```
save space when cloning OpenMS
Adapter has been removed
and add test for xsd use in XMLValidator
and remove some code that was needed for tools that are now deprecated
@bernt-matthias bernt-matthias marked this pull request as ready for review June 13, 2024 14:56
@bernt-matthias
Copy link
Collaborator Author

Linter errors are all: WARNING (TestsMissing): No tests found, most tools should define test cases. which won't be fixed and will be ignored for deployment.

Copy link
Member

@bgruening bgruening left a comment

Choose a reason for hiding this comment

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

Nothing scary in here, added 3 small comments, but from my side this is good. Very cool, thanks!

@bernt-matthias
Copy link
Collaborator Author

Ready :)

@bgruening bgruening merged commit 5c080b1 into galaxyproteomics:master Jun 14, 2024
12 of 14 checks passed
@bgruening
Copy link
Member

🥳🥳🎉🎉

@bernt-matthias bernt-matthias deleted the openms2.9 branch June 14, 2024 18:41
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.

2 participants