feat: v0.3.0-alpha feedback#21
Conversation
…hist. rework hist implmentation to be identical to matplotlib. rework the demo notebook to be simpler. Add better errors in backend.
marcboulle
left a comment
marcboulle
left a comment
There was a problem hiding this comment.
Tout cela me semble très bien
Quelques améliorations de détail indiquées en note de review
Pour le derniers points de commit: ne garder qu'un titre court, et mettre en note de commit ce qui a été fait (essentiellement, reprendre le dernier commentaire que tu as ajouté dans l'issue feedback).
Le nouvelle version de la demo me semble très bien, mais c'est difficile de se rendre compte dans l'interface de review de github: j'aurais besoin d'une béta 2 du site pour tout relire une dernière fois (après résolution du problème Sphynx résiduels avec Vladimir).
ElouenGinat
changed the title
| """ | ||
| x = np.asarray(x, dtype=np.float64) | ||
|
|
||
| if len(x) == 0: |
There was a problem hiding this comment.
I would use the NumPy ndarray API here: if x.size == 0:.
|
|
||
| x = x[~np.isnan(x)] | ||
|
|
||
| if len(x) == 0: |
| a, | ||
| range=None, | ||
| max_bins=None, | ||
| density=False, |
There was a problem hiding this comment.
Change to density=True, as this is the default value now.
| x, | ||
| range=None, | ||
| max_bins=None, | ||
| density=False, |
popescu-v
left a comment
popescu-v
left a comment
There was a problem hiding this comment.
LGTM, with a few minor comments.
3 participants
No description provided.