Numpy: Crash dans `np.vdot` pour un objet de type tableau

Hmm, vdot vérifie ce que le dtype de sortie sera à l'avance (ce qui, je suppose, est très bien). Cependant, cette fonction peut échouer et renvoyer NPY_NOTYPE , elle devrait donc renvoyer une erreur lorsque NPY_NOTYPE est renvoyé. C'est dans https://github.com/numpy/numpy/blob/master/numpy/core/src/multiarray/multiarraymodule.c#L2241 (et la ligne suivante), l'erreur que vous voyez est un retour d'erreur plus tard, mais je pense que cette fonction ne devrait vraiment pas renvoyer d'erreur (même si ce serait bien d'ajouter une vérification d'erreur là aussi).

Voulez-vous faire un PR?

(Cela pourrait être bien de vérifier s'il existe d'autres utilisations similaires où la vérification des erreurs est manquante.)

Salut,
Je suis un nouveau venu et j'aimerais travailler sur cette question. Des pistes sur par où commencer?
Merci d'avance.

Découvrez notre workflow de développement . Ensuite, vous devriez arriver à un point où vous pouvez exécuter python runtests.py sur la branche master ( git checkout master; git clean -xfd; python runtests.py ).

Lorsque vous êtes configuré, vous pouvez commencer à pirater: créez une branche, ajoutez un test qui échoue ( numpy/core/tests/teste_multiarray.py dans TestVDot serait un bon endroit), essayez de corriger le code C à partir de l'indice ci-dessus . Si vous êtes bloqué, n'hésitez pas à demander de l'aide, plus vous pourrez montrer que vous avez fait un véritable effort pour suivre la documentation et les conseils, mieux ce sera.

Bienvenue @ Soniyanayak51!

Pour ajouter à ce que @mattip a dit: une petite information qui manque dans nos documents (je pense) est la façon dont nous travaillons sur les problèmes. Nous n'utilisons pas la fonctionnalité "assignés" de GitHub, donc à moins qu'il ne soit clair d'après les récents commentaires / relations publiques que quelqu'un travaille activement sur un problème, vous pouvez simplement vous y plonger.

@rgommers @mattip J'ai ajouté des vérifications pour les types obtenus à partir de PyArray_DescrFromType (). Je ne suis pas sûr de devoir les ajouter également pour PyArray_DESCR ().
De plus, à propos de l'ajout de tests dans test_multiarray.py, dois-je ajouter un test_PyArrayType et ajouter des tests où le type est NULL? Je ne sais pas comment aller de l'avant avec l'ajout de tests.

@rgommers @mattip J'ai ajouté des vérifications pour les types obtenus à partir de PyArray_DescrFromType (). Je ne suis pas sûr de devoir les ajouter également pour PyArray_DESCR ().
De plus, à propos de l'ajout de tests dans test_multiarray.py, dois-je ajouter un test_PyArrayType et ajouter des tests où le type est NULL? Je ne sais pas comment aller de l'avant avec l'ajout de tests.

On dirait que @mattip a répondu à cela en partie sur le PR (je pense). Il vaut probablement mieux garder la conversation là-bas.

dois-je ajouter un test_PyArrayType et ajouter des tests où le type est NULL?

Pour ajouter un peu: le principe pour ajouter des tests est normalement de ne tester que l'API publique. Alors tu prendrais

...
np.vdot(Foo(), Foo())

à partir de la description du problème ci-dessus, et ajoutez-le comme test. Il ne semble pas nécessaire de tester l'API C séparément.