-
Notifications
You must be signed in to change notification settings - Fork 58
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
PS updated to 2.0.15 & SG updated to 4.0.22 #570
Merged
bernt-matthias
merged 16 commits into
galaxyproteomics:master
from
CarlosHorro:sg_4022_ps_2014
Mar 16, 2021
Merged
Changes from 5 commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
17491b3
PS updated to 2.0.14
CarlosHorro 7700dc1
Removed wrong result in test (removed from PS after last update).
CarlosHorro e9a71c4
Merge branch 'master' into sg_4022_ps_2014
CarlosHorro a254773
PS updated to 2.0.15
CarlosHorro a9e4f37
use_shared_home="false"
CarlosHorro 93f1d0a
some fixes for searchgui and peptideshaker
bernt-matthias a6e9a89
Profile updated to 20.01
CarlosHorro ac36119
Added logs to stderr output.
CarlosHorro d7c9057
Added logs to PS stderr output.
CarlosHorro 95736b3
detect oom errors as fatal_oom and not fatal
bernt-matthias 7416940
Merge pull request #2 from bernt-matthias/CarlosHorro_sg_4022_ps_2014
CarlosHorro fe669cf
Merge branch 'master' into sg_4022_ps_2014
CarlosHorro 48797cf
Undone some last features until app behaviour is fixed.
CarlosHorro 2bc4ae3
Refreshing PR
CarlosHorro bbe7a74
Added profile 20.01 to fasta_cli and ident_params tools
CarlosHorro f921b21
Readded PS 20.01 profile.
CarlosHorro File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better to increase the
profile
of the tools to something greater than or equal to 18.01.See https://github.com/galaxyproject/galaxy/blob/ce23f963e036722865ecd27fda509f173406ab61/lib/galaxy/tool_util/parser/xml.py#L169
I would suggest at least 20.01
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you are right @bernt-matthias, thank you. That was something I wanted to do some time ago but I didn't remember to include it...