Closed twojenski closed 8 years ago
Hi Tek,
Thanks for telling us about this. Would it be possible for you to send us the old parameter file so that we can try to reproduce and hopefully fix the issue?
Best regards, Harald
Hello Harald
Sure. Here are 2 .par files that will not work. Essentially the newly introduced attribute proteinInferencePreferences (see below) never get initialized properly in code because they do not exist in the old .par files. This causes the NULL Pointer Exception. Please note that this may only be the first attribute that causes a problem, there could be others once this one is fixed.
"proteinInferencePreferences": { "proteinSequenceDatabase": { "path": "C:\Work\2\2\ABC.fasta" } },
Regards
Tek Wojenski
On Wed, Feb 10, 2016 at 4:31 PM, Harald Barsnes notifications@github.com wrote:
Hi Tek,
Thanks for telling us about this. Would it be possible for you to send us the old parameter file so that we can try to reproduce and hopefully fix the issue?
Best regards, Harald
— Reply to this email directly or view it on GitHub https://github.com/compomics/compomics-utilities/issues/10#issuecomment-182589851 .
Hi Tek,
It seems like you tried to include the parameter files as attachment to your e-mail? I'm afraid this is not supported by GitHub. You'll have to log on to the issue tracker and upload the two files there.
Best regards, Harald
Here they are, rename PDF to ZIP first. I could not upload .ZIP???
Hi Tek,
Thanks for the test files. The problem was that your parameter file came from DeNovoGUI and DeNovoGUI, for obvious reasons, does not have a FASTA file as part of its parameters. Which made it a bit difficult for us to show the name of the database in the dialog. :)
This has now been corrected and you should be able to load your parameter files in the next releases of SearchGUI and PeptideShaker, hopefully out tomorrow.
Best regards, Harald
Hi Tek,
The issue should be fixed with the new releases of SearchGUI, PeptideShaker and DeNovoGUI. Please let us know if there are other old parameter files you cannot load, as there might be some other special cases like this that we have not yet detected.
Best regards, Harald
I found a problem when I'm trying to import/parse in older .par files into PeptideShaker.
The actual problem is not with deserializing old .par files which could be either JSON or Binary. The problem occurs in IdentificationParametersEditionDialog on line 285 when calling: proteinInferencePreferences.getShortDescription() It seems that ProteinInferencePreferences is a new attribute, and is never properly initialized by the old .par files. When updateGUI() is called it gets a null Pointer when processing: proteinInferencePreferences.getShortDescription().
Regards
Tek Wojenski