oleksiiparf / plsnpairs

Automatically exported from code.google.com/p/plsnpairs
0 stars 0 forks source link

Prediction vs PC # Plot Tool improvements #141

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Issues:
(i) Error message (if 'subj_labels', now called 'split_obj_labels', doesn't 
exist) needs newline at end.
(ii) When saving text file associated with this plot, check whether filename 
supplied by user has suffix '.txt'; add this suffix automatically if it wasn't 
supplied
(iii) Change 'Variable range' label to 'PC range' (also applies to Rep vs Pred 
tool)
(iv) GUI needs various other improvements (this applies equally to Rep vs Pred 
Plot in Other Tools since they use same interface): adding new groups, e.g., is 
awkward; having to specify entire results file path + full filename pattern is 
also awkward (maybe separate dir and file and allow user to browse for dir 
path?)... need to consider this in greater detail.

Original issue reported on code.google.com by anitaeo...@gmail.com on 14 Sep 2010 at 6:36

GoogleCodeExporter commented 8 years ago
Changes were made r1480 Sept 16.  (i) and (iii) are good.  

(ii): suffix ".txt" is always added, even if file already ends in ".txt".  
Should check if suffix already exists and only add it only if it doesn't.  

(iv): Some more proposed changes:
a. Change 'Group' everywhere to 'Curve'.
b. Change 'Apply changes to this Group [Curve]' button to just 'Apply' and move 
it below interactive fields.
c. Change 'Add new group [curve] and save current' to just 'Add new [curve]' 
and save current automatically.
d. Add 'Clear' button next to new 'Apply' button so current text can be cleared 
from screen.
e. Disable 'Generate Plot' button until at least one Curve has been 'Applied' 
(or loaded from file).

Original comment by anitaeo...@gmail.com on 28 Sep 2010 at 8:02

GoogleCodeExporter commented 8 years ago
These changes can be found in revision 1579

Original comment by flt.john...@gmail.com on 29 Sep 2010 at 7:36