docs(guides): expliquer densité et largeur des classes - #31
Conversation
|
marcboulle
left a comment
marcboulle
left a comment
There was a problem hiding this comment.
L'explication sur la densité me parait très bien:
- quelques suggestion de détail
- à valider avec les utilsateurs, si possible
Installation sur Windows: résoudre si possible
Passage à la 1.0.2: prématuré, attendre que les autres problèmes connus soient résolus
| within that interval. When bin widths vary, height alone is not enough: | ||
| probability is represented by the bar's area, calculated as density multiplied | ||
| by bin width. | ||
|
|
There was a problem hiding this comment.
For a detailed introduction to density and variable-width bins
->
Note that variable-width bins are ideal for unbalanced distributions and outliers, as shown in [Histograms - Khiops].
There was a problem hiding this comment.
I added back this section. I had a reviewer telling me he liked it (it's maybe a little too long...
What are your recommandations?
There was a problem hiding this comment.
L'utilisateur est clairement prioritaire; il faut donc garder ce tableau.
Par contre, corriger:
"Counts divided by 𝑁
, or density"
=>
"Counts divided by 𝑁
, or density times bin width"
| On Windows, install [Visual Studio Build Tools 2022](https://visualstudio.microsoft.com/visual-cpp-build-tools/) | ||
| with the **Desktop development with C++** workload. Then run the commands below | ||
| from an **x64 Native Tools Command Prompt for VS 2022** and verify that the | ||
| compiler is available with `where cl`. |
There was a problem hiding this comment.
Ce serait vraiment un plus de résoudre le problème d'installation sur Windows.
Sur mon poste, j'ai un Visual Studio installé, et je n'ai aucun problème avec le repo khiops concernant le compilateur C++, alors que j'ai eu des erreurs avec khisto-python.
There was a problem hiding this comment.
Can you retry on your desktop plz ?
There was a problem hiding this comment.
En me plaçant sur la branche de la PR et en relançant uv sync --group dev --extra all, je n'ai plus d'erreur, et le khisto.exe est compilé dans .venv\Scripts.
Par contre, le where cl me renvoie Information : impossible de trouver des fichiers pour le(s) modèle(s) spécifié(s).
Soit on résout ce nouveau problème, soit on vire la fin de phrase and verify that the compiler is available withwhere cl`, peut-être pas très pertinente ici.
marcboulle
left a comment
marcboulle
left a comment
There was a problem hiding this comment.
Quelques petits détails à régler (cf. commentaires).
Plus remarques concernant la partie "Counts vs density":
- le titre est sur deux lignes, alors que j'ai largement la place sur mon écran
- il y a une barre verticale de pointillés entre les figures de gauche et celle de droite, qui coup le libellés de l'axe des Y des figures de droite
| # Publish to GH pages on Git tag push | ||
| preview: | ||
| if: >- | ||
| always() && |
There was a problem hiding this comment.
What does always() add here, as it is always true? I would consider removing it.
There was a problem hiding this comment.
always() permet d’exécuter ce job lorsque build est skipped, notamment à la fermeture d’une PR. Sans lui, GitHub Actions applique implicitement success() et ignore preview, empêchant ainsi la suppression de l’aperçu. Je propose donc de le conserver.
3 participants
What
Why
Clarifier l’interprétation des histogrammes adaptatifs et éviter de comparer directement les hauteurs d’effectifs lorsque les classes ont des largeurs différentes.
How to test
Le rendu a également été vérifié en modes clair et sombre, sur desktop et mobile.