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

Some IdentificationParmetersCLI parameters and expected options are different from documentation #186

Closed
Thys3Potgieter opened this issue Jun 25, 2016 · 8 comments
Assignees
Labels

Comments

@Thys3Potgieter
Copy link

Hi Guys, im sure that this rather a documentation issue and not a bug.

$ java -cp PeptideShaker-1.11.0.jar eu.isas.peptideshaker.cmd.IdentificationParametersCLI
-sequence_matching_type '2' -out /home/user/example

Gives:
Error parsing the sequence_matching_type option: Found 2. Supported input: [Protein, Peptide, Spectrum, Ion, PTM].
Run -usage to see the list of supported options and their input.

But the documentatation (http://compomics.github.io/compomics-utilities/wiki/identificationparameterscli.html):

-sequence_matching_type The peptide to protein sequence matching type. Default is 2.
0: Character Sequence
1: Amino Acids
2: Indistinguishable Amino Acids

Simarly, the "-usage" command also states that a number between 0-2 is expected.

The new and old parameters also seem very different,

Kind regards

Thys

@Thys3Potgieter Thys3Potgieter changed the title IdentificationParmetersCLI sequence_matching_type options are different from documentation IdentificationParmetersCLI sequence_matching_type expected options are different from documentation Jun 25, 2016
@Thys3Potgieter Thys3Potgieter changed the title IdentificationParmetersCLI sequence_matching_type expected options are different from documentation IdentificationParmetersCLI "sequence_matching_type" expected options are different from documentation Jun 25, 2016
@Thys3Potgieter
Copy link
Author

Another one:

"Error parsing the ptm_sequence_matching_type option: Found 1. Supported input: [Protein, Peptide, Spectrum, Ion, PTM]."

@Thys3Potgieter
Copy link
Author

Thys3Potgieter commented Jun 25, 2016

And this one (I am testing them all because I am sourcing all the settings from bash config file on a linux hpc cluster...and mostly passing the default settings for now):

eg.

java -cp /home/ptgmat003/PeptideShaker-1.11.0/PeptideShaker-1.11.0.jar eu.isas.searchgui.cmd.IdentificationParametersCLI -xtandem_output_results 'all' -out '/home/user/test'

java.lang.NumberFormatException: For input string: "all"
at sun.misc.FloatingDecimal.readJavaFormatString(FloatingDecimal.java:1241)
cjava -cat java.lang.Double.valueOf(Double.java:504)2.9.0.jar eu.isas.searchgui.cmd.IdentificationParametersCLI -xtandem_output_results ionParametersCli.java:92)
at java.lang.Double.(Double.java:597)metersCLI.ejava -cp /home/ptgmat003/SearchGUI-2.9.0/SearchGUI-2.9.0.jar eu.isas.searchgui.cmd.IdentificationParametersCLI -xtandem_output_results '
at com.compomics.util.experiment.identification.parameters_cli.IdentificationParametersInputBean.updateIdentificationParameters(IdentificationParametersInputBean.java:549)
'all' -oat com.compomics.util.experiment.identification.parameters_cli.IdentificationParametersInputBean.(IdentificationParametersInputBean.java:118)
at com.compomics.util.experiment.identification.parameters_cli.AbstractIdentificationParametersCli.initiate(AbstractIdentificationParametersCli.java:92)
at eu.isas.searchgui.cmd.IdentificationParametersCLI.(IdentificationParametersCLI.java:25)
at eu.isas.searchgui.cmd.IdentificationParametersCLI.main(IdentificationParametersCLI.java:36)

Docs:
-xtandem_output_results X!Tandem 'output, results' option (all|valid|stochastic), default is 'all'.

@Thys3Potgieter Thys3Potgieter changed the title IdentificationParmetersCLI "sequence_matching_type" expected options are different from documentation Some IdentificationParmetersCLI parameters and expected options are different from documentation Jun 25, 2016
@Thys3Potgieter
Copy link
Author

java -cp /home/ptgmat003/SearchGUI-2.9.0/SearchGUI-2.9.0.jar eu.isas.searchgui.cmd.IdentificationParametersCLI -myrimatch_isotope_low '-1' -out '/home/test/user'

Unrecognized option -myrimatch_isotope_low.

Run -usage to see the list of supported options and their input.

Docs:
-myrimatch_isotope_low MyriMatch lower isotope error range, default is '-1'.

@Thys3Potgieter
Copy link
Author

Unrecognized option -myrimatch_isotope_high.

Run -usage to see the list of supported options and their input.

Unrecognized option -msgf_isotope_low.

Run -usage to see the list of supported options and their input.

Unrecognized option -omssa_isotopes.

@Thys3Potgieter
Copy link
Author

java -cp /home/ptgmat003/SearchGUI-2.9.0/SearchGUI-2.9.0.jar eu.isas.searchgui.cmd.IdentificationParametersCLI -msgf_isotope_high '1' -out '/home/test/user'

Unrecognized option -msgf_isotope_high.

Run -usage to see the list of supported options and their input.

Docs:
-msgf_isotope_high MS-GF+ upper isotope error range, default is '1'.

@Thys3Potgieter
Copy link
Author

Last one:
Command:
java -cp /home/ptgmat003/SearchGUI-2.9.0/SearchGUI-2.9.0.jar eu.isas.searchgui.cmd.IdentificationParametersCLI -omssa_it_replace_evalue '0.0' -out '/home/test/user'

Output:
"Error parsing the omssa_it_replace_evalue option: Negative or zero value found.
Run -usage to see the list of supported options and their input."

Docs:
-omssa_it_replace_evalue OMSSA e-value cutoff to replace a hit in the iterative search 0.0 means keep best. Default is '0.0'.

@hbarsnes hbarsnes self-assigned this Jun 25, 2016
@hbarsnes hbarsnes added the bug label Jun 25, 2016
@hbarsnes
Copy link
Member

Hi Thys,

Thanks for pointing these out.

The "sequence_matching_type" and "ptm_sequence_matching_type" were both due to the same bug in which two parameters had gotten mixed up in the parameter check.

The same was the case for "xtandem_output_results".

The search engine specific isotope options you refer to have all been replaced by the generic options "min_isotope" and "max_isotope" which controls the isotope range across the search engines.

Finally, the "omssa_it_replace_evalue" issue was due to the code not allowing zero values. Which is a bit stupid when the default value is 0.0. ;)

All now fixed and new versions of both SearchGUI and PeptideShaker have been released.

I'm closing this issue, but please don't hesitate to open a new one if you find other errors in the command line documentation.

Best regards,
Harald

@Thys3Potgieter
Copy link
Author

Thanks so much for a speedy reply! I will download the latest version :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants