Skip to content
Snippets Groups Projects
Commit b83acde8 authored by Steve Reis's avatar Steve Reis
Browse files

fix(exareme): Matrix order in PCA Algorithm result

parent 8ff57872
No related branches found
No related tags found
No related merge requests found
...@@ -84,7 +84,10 @@ describe('PCA result handler', () => { ...@@ -84,7 +84,10 @@ describe('PCA result handler', () => {
exp.results.forEach((it) => { exp.results.forEach((it) => {
if (it['matrix']) { if (it['matrix']) {
const heatmap = it as HeatMapResult; const heatmap = it as HeatMapResult;
expect(heatmap.matrix).toEqual(data.eigen_vecs); const matrix = data.eigen_vecs[0].map(
(_, i) => data.eigen_vecs.map((row) => row[i]), // reverse matrix as we want row-major order
);
expect(heatmap.matrix).toEqual(matrix);
expect(heatmap.yAxis.categories).toEqual(exp.variables); expect(heatmap.yAxis.categories).toEqual(exp.variables);
} }
}); });
......
...@@ -42,7 +42,13 @@ export default class PCAHandler extends BaseHandler { ...@@ -42,7 +42,13 @@ export default class PCAHandler extends BaseHandler {
}, },
}; };
if (matrix) exp.results.push(heatMapChart); if (matrix && matrix.length > 0) {
heatMapChart.matrix = matrix[0].map(
(_, i) => matrix.map((row) => row[i]), // reverse matrix as we want row-major order
);
}
if (heatMapChart.matrix) exp.results.push(heatMapChart);
this.next?.handle(exp, data); this.next?.handle(exp, data);
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment