Numpy: Document ndindex (forme)

Créé le 3 juil. 2020  ·  3Commentaires  ·  Source: numpy/numpy

Cette utilisation de ndindex :

>>> list(np.ndindex((3,2)))
[(0, 0), (0, 1), (1, 0), (1, 1), (2, 0), (2, 1)]

semble fonctionner avec

>>> np.__version__
'1.18.4'

mais ce n'est pas documenté . Cette utilisation est-elle censée être autorisée ? (J'aimerais que ce soit le cas, car si shape == 5 alors np.index(shape) fonctionne alors que np.ndindex(*shape) ne fonctionne pas.) Si c'est le cas, pourrait-il être ajouté à la documentation ?

04 - Documentation

Commentaire le plus utile

De plus, la liste des types dans la section Paramètres est incorrecte : les tuples et la liste d'entiers sont autorisés.

Cela a été corrigé.

La signature de la docstring np.ndindex est incohérente avec la liste des paramètres

Je ferai un PR pour ça :)

Tous les 3 commentaires

Nous voudrons peut-être faire de cette version la version documentée préférée, et mentionner brièvement l'autre option également. Dans la plupart des cas (par exemple aussi arr.reshape() où nous autorisons les deux, nous préférons la version tuple dans la documentation).

Juste pour clarifier un peu cela - je pense qu'il y a deux choses qui rendent la docstring np.ndindex déroutante :

  1. La signature de la docstring np.ndindex est incohérente avec la liste des paramètres
  2. De plus, la liste des types dans la section Paramètres est incorrecte : les tuples et la liste d'entiers sont autorisés.

De plus, la liste des types dans la section Paramètres est incorrecte : les tuples et la liste d'entiers sont autorisés.

Cela a été corrigé.

La signature de la docstring np.ndindex est incohérente avec la liste des paramètres

Je ferai un PR pour ça :)

Cette page vous a été utile?
0 / 5 - 0 notes