poppa10 / plsnpairs

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

Session file compatibility issues between PLS/NPAIRS/Matlab/Java #107

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
- See attached spreadsheet delineating session file save/load behaviour across 
PLS/NPAIRS/Matlab/Java
- TODO: (i) revise session window to force users to save session file every 
time datamat is created and to use same prefix for sess  and datamat files (as 
is required in ML plsgui);
(ii) handle NullPointerException crash described in spreadsheet (happens when 
PLS Block Java file with no mask file info is loaded into NPAIRS Block session 
file window) by printing Warning to user that mask file is required for NPAIRS 
Block so info must be added and session file resaved; note that if PLS Matlab 
sess file (which contains no mask info ever) is loaded, a warning *is* issued 
gracefully, so it's only when sess files containing extra Java variables 
'scans_skipped' and 'runs_skipped' but no mask variable that the crash occurs;
(iii) think more about how compatibility between NPAIRS and PLS Java and ML 
session files should be configured: 
- Should mask thresh info also be saved in sess files so it can eventually be 
used by NPAIRS Block when automatic mask thresholding is added to NPAIRS Block? 
- Should mask/skip scans/ignore runs info be moved out of sess files altogether 
and stored in a separate file?  
- Is there ever any need to save this extra information in event-related NPAIRS 
or PLS session files?  
- Note that Merge Across Runs is always set to true when NPAIRS Block sess file 
loaded in PLS (ML or Java).  Should user be warned about this?  

Original issue reported on code.google.com by anitaeo...@gmail.com on 29 Jul 2010 at 12:21

Attachments:

GoogleCodeExporter commented 8 years ago
Some more details:

Currently, mask/skipped scan/ignored run info is stored in session file (in 
Java) to ensure that Block PLS (Java) session files are totally compatible with 
NPAIRS Block, since NPAIRS Block uses no datamats but PLS (Matlab and Java) and 
NPAIRS Event-related (Java) store this info in the datamat file. Since ML 
session files never contain mask/skipped scan/ignored run info, it is not 
currently possible to use them in Block NPAIRS without resaving them first in 
Java with the additional required info.

Original comment by anitaeo...@gmail.com on 29 Jul 2010 at 12:24

GoogleCodeExporter commented 8 years ago
At v 1.1.10, almost all compatibility and usability issues have been resolved.

Re TODO list reported Jul 28, 2010:
(i) session prefix forced to be same as datamat prefix (as of v 1.1/1.1.1);
(ii) no crashes in Java GUI anymore when loading ML- or Java-generated sess 
files of any type into any window; warnings printed when missing info;
(iii) 
- mask thresh info is stored in sess files;
- mask/skip scans/ ignore runs info still in session file, including in 
event-related NPAIRS and PLS session files (and loaded automatically into Java 
GUI, though not recognized in ML); however, see below for more about skipped 
scans;
- user is not warned about Merge Across Runs automatically being set; should 
they be? MAC is default choice in ML, too, so perhaps not.

Remaining compatibility issues between session files are detailed in attached 
spreadsheet:
- 4d num scans recognition problem from Java-generated sess files;
- num skipped scans no longer recognized in ML from Java-gen files;
- norm info in datamat setup delselected when NPAIRS block file loaded in Java 
PLS block window (what happens when datamat generation is run?).

Original comment by ao...@research.baycrest.org on 23 Jan 2013 at 5:46

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by anitaeo...@gmail.com on 23 Jan 2013 at 5:47

GoogleCodeExporter commented 8 years ago

Original comment by anitaeo...@gmail.com on 23 Jan 2013 at 5:58

GoogleCodeExporter commented 8 years ago
Note that as of V5.1105021, PLS Matlab no longer has 'runs to be included' 
option: Here is relevant part of PLS Matlab software update documentation (May 
2, 2011, from http://www.rotman-baycrest.on.ca/pls/whatsnew.txt):

'"Run to be included" field is removed. If you want to exclude the run, simply 
open "Edit Runs" window, go to the run that you need to delete, and click 
Edit->Delete."'

Ignore runs info is now used only in PLSNPAIRS! 

See updated sess file compatibility doc.

Original comment by anitaeo...@gmail.com on 23 Jan 2013 at 7:28

Attachments:

GoogleCodeExporter commented 8 years ago
See also Issue # 192.

Original comment by anitaeo...@gmail.com on 23 Jan 2013 at 8:09

GoogleCodeExporter commented 8 years ago
See also Issue # 202

Original comment by anitaeo...@gmail.com on 26 Aug 2014 at 8:11

GoogleCodeExporter commented 8 years ago
Need to add sessiondata.mat files into this spreadsheet.

Original comment by ao...@research.baycrest.org on 27 Aug 2014 at 7:53

GoogleCodeExporter commented 8 years ago
Compatibility v1.1.19: ML session/datamat files include ML-generated 
sessiondata files. 

Original comment by ao...@research.baycrest.org on 27 Aug 2014 at 9:03

Attachments: