onnx/onnx

Why does SplitToSequence not allow zeros in `split` input?

Open

#6 356 ouverte le 9 sept. 2024

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)Python (3 585 forks)batch import
contributions welcomegood first issuemodule: specquestion

Métriques du dépôt

Stars
 (16 592 stars)
Métriques de merge PR
 (Merge moyen 1j 23h) (80 PRs mergées en 30 j)

Description

Question

The ONNX 1.18.0 documentation says that for operator SplitToSequence:

‘split’ must contain only positive numbers.

Why are zeros not allowed? The semantics for zeros seems unambiguous, and it seems like they should be allowed per the principle of least surprise.

Guide contributeur

Direction de recherche
Examiner le fichier de spécification ONNX pour l'opérateur SplitToSequence dans le répertoire `onnx/defs`. Vérifier la définition de l'opérateur et la contrainte de type qui interdit les zéros. Consulter des opérateurs similaires pour justification. Envisager de proposer un changement pour autoriser les zéros et mettre à jour la documentation en conséquence.
Stack technique
python
Domaine
machine learningapi
Type d'issue
Recherche
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
understanding of ONNX operators
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65