TPP:V5Feedback

From SPCTools

Revision as of 19:26, 17 July 2015 JoeS (Talk | contribs) (→Release 5.0.0a2) ← Previous diff		Revision as of 19:28, 17 July 2015 JoeS (Talk | contribs) (→Release 5.0.0a2) Next diff →
Line 11:		Line 11:
	* <s>Adjust web timeout!?? Seemed to have pooped at 2 minutes...? Done</s>		* <s>Adjust web timeout!?? Seemed to have pooped at 2 minutes...? Done</s>
	* PepXMLViewer: BAD link to SpectraST ("ST")		* PepXMLViewer: BAD link to SpectraST ("ST")
-	* Petunia icon on desktop and all pages!	+	* Petunia icon on desktop
	** Window's quirk? Sometimes I see this sometimes I don't. The code in the installer looks solid.		** Window's quirk? Sometimes I see this sometimes I don't. The code in the installer looks solid.
		+	* Petunia icon in browser tab is missing on some pages
	* tpp_models: bad links to js/ resources ("/ISB/html...")		* tpp_models: bad links to js/ resources ("/ISB/html...")
	** missing jquery.flot.errorbars.js		** missing jquery.flot.errorbars.js

---

Revision as of 19:28, 17 July 2015

Feedback related to releasing TPP version 5.0

This page here is to record feedback/issues found with the development builds of 5.0.0

Release 5.0.0a2

• Uninstall leaves cmd.exe, and data/ log/ users/ dirs. (by design?)
  • Joe: Yes. Well at least data/, log/. Don't think you'd want to remove TPP and lose all your data. The users/ directory could get removed, and I guess the log directory if you guys think so. The cmd.exe file, is that in bin? If so it should have been removed.
• ERROR in Petunia. CGI::multi_param is not a valid subroutine (new version of CGI.pm)
  • This again?!?! So if you use ActivePerl you'll need to update CGI, I sent along instructions in a email to Luis. Otherwise yes, there still are places where parm() is called in a array context and needs to be replaced with multi_param(). I did another pass looking for these occurrences, but there are many. Someone other than me should do a grep 'param(' and check them out.
• Adjust web timeout!?? Seemed to have pooped at 2 minutes...? Done
• PepXMLViewer: BAD link to SpectraST ("ST")
• Petunia icon on desktop
  • Window's quirk? Sometimes I see this sometimes I don't. The code in the installer looks solid.
• Petunia icon in browser tab is missing on some pages
• tpp_models: bad links to js/ resources ("/ISB/html...")
  • missing jquery.flot.errorbars.js
• ProtXMLViewer fails...(bad gateway)
• Old ProtXMLViewer fails too: system cannot find the path specified
• updatepaths does not seem to do anything, no output, even with -vv
• mzIdent conversion fails
• Petunia::Issue when selecting a directory from quick-select drop-down
• decoyFastaGenerator.pl does not launch from Petunia
• Downloading Spectral Libs fails, yet again: bad paths to md5sum, wget, and unzip

Release 5.0.0a1

• Merge last set of trunk changes into branch

• Review one last time what's in external. Ditch unneeded stuff.

• Add instructions on wiki on how to use your own Perl/Apache
  • (Joe) warn users if they choose not to install perl or appweb

• Luis says the TPP icon is ugly and use the purple flower instead

• Update linux scripts in installer_linux

• Update BUILD_LINUX. Especially Apache instructions

• Luis says msconvert fails.
  • Luis is right<\s>

• <s>README opens poorly in Notepad (needs DOS line endings)

• Move tools params from /conf to data/param(eter)s/
  • They need to be accessible from Petunia/web browser
  • Which files? taxonomy.xml and tandem_params.xml only? or more?
• Attempted to access forbidden directory error when selecting files and adding them. It's comparing $data_dir with param('workingdir'). One has a '/' on it and one doesn't.

• Keep an eye out for "this page failed" in Petunia when submitting forms. If prompted resend try it a couple of times. If the page appears check the bottom of the page out. There may be a message about security and the params() function. If so note the line #. I've gone through tpp_gui.pl trying to fix all of these calls but apparently have missed some.

• Create data/dbase/speclibs/ dir, if not present

• PWiz -lrt issue on Windows

• Upgrade comet

• Upgrade pwiz-win32