multi comparison (matrix, cube) #15
Conversation
|
Is there any check that all the plots provided belong to the same type? So that we do not proceed with let's say one plot for cube and two plots for square at the same time. |
added this check (the first json determines the plot type, the rest must match) |
|
@stan-donarise please could you move |
|
@nin-jin looking fwd for your (very critical) review 🥇 |
lib/pca/pca.ts
Outdated
| @@ -0,0 +1,5 @@ | |||
| namespace $ { | |||
|
|
|||
| export const $mpds_visavis_lib_pca = require('../mpds/visavis/lib/pca/bundle/pca.js') | |||
There was a problem hiding this comment.
🟠 Какая-то странная ссылка. Она тут должна быть относительно текущего файла.
Возможно лучше было бы переименовать pca.js в _pca.js (чтобы сборщик сам её не включал в бандл) и положить рядом, а потом подключит как-то так: require('./_pca.js')
There was a problem hiding this comment.
вроде сборщик устроен так, что у require должны быть полные пути?
т.е. например в бандле (web.js) генерируется такой код $node[ "../mpds/visavis/lib/pca/_pca.js" ]
а вот относительно текущего файла require('./_pca.js') - так не работает
There was a problem hiding this comment.
Но полные пути должны с / начинаться. Выход из корня наверх не должен работать.
There was a problem hiding this comment.
вот https://github.com/stan-donarise/mam_mol/blob/master/build/build.node.ts#L937C1-L937C11
видимо из node_modules поднимается
No description provided.