khiops icon indicating copy to clipboard operation
khiops copied to clipboard

Numerical Variable Encode with bad label value

Open cfkstat opened this issue 1 year ago • 2 comments

Description

]-inf;21.5] => (-inf, 21.5] ]30.5;33.5] => (30.5, 33.5]

Context

image

cfkstat avatar Nov 17 '24 05:11 cfkstat

Imho, the two notations are used. But indeed, the value separator should be ‘,’ instead of ‘;’.

cf. https://en.wikipedia.org/wiki/Interval_(mathematics) Including or excluding endpoints To indicate that one of the endpoints is to be excluded from the set, the corresponding square bracket can be either replaced with a parenthesis, or reversed. Both notations are described in International standard ISO 31-11. Cf. https://en.wikipedia.org/wiki/Interval_(mathematics)#Notations_for_intervals

marcboulle avatar Nov 18 '24 16:11 marcboulle

Pour remplacer le séparateur ";" par ",", la modification est minime : 2 lignes impactés dans KWDGInterval::GetObjectLabel() Cela ne modifie aucun résultat de TestKhiops ou TestKNI : uniquement les résultats de TestCoclustering

Il y a par contre un impact à gérer dans KhiopsCovizualisation car, avec ce nouveau séparateur, le repliement des intervalles de variables numériques de coclustering ne fonctionne plus

Image

carinehue avatar Feb 21 '25 13:02 carinehue