josecamachop / MEDA-Toolbox

Multivariate Exploratory Data Analysis Toolbox for Matlab
GNU General Public License v3.0
30 stars 17 forks source link

Mdarmstr paraglm octave compatibility #51

Closed mdarmstr closed 1 year ago

mdarmstr commented 1 year ago

Since Octave doesn't have table functionality, this is just a suggestion to create T as a structure containing the same information.

josecamachop commented 1 year ago

Hi, thanks for spotting the error.

I prefer the use of the table, because it reflects the approach in ANOVA routines in matlab. In general, for compatibility issues we may adopt the solution in

https://stackoverflow.com/questions/2246579/how-do-i-detect-if-im-running-matlab-or-octave

isOctave = exist('OCTAVE_VERSION', 'builtin') ~= 0;

Finally, there is a number of parglm routines and all of them should apply the same solution.

josecamachop commented 1 year ago

This is finally solved in commit https://github.com/josecamachop/MEDA-Toolbox/commit/3650fd93558ae2b77340505842cb0a17cd0e1716, which is actually inspired in your solution