ucb-bar/onnxruntime-riscv

Refactoring: Use the inbuilt im2col NHWC?

Ouverte

#26 ouverte le 27 janv. 2021

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C++ (37 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (93 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

More of a code cleanliness thing, but currently for im2col in nhwc layout we use our own method that I took from the pytorch docs. ONNXRuntime also includes its own im2col nhwc implementation (see the cpu math ops). The two functions are basically identical except in our own im2col_nhwc function it does it for all groups whereas the ORT one only does it for one group. So when you refactor you'll have to move the im2col inside the for-loop where we iterate over the groups.

Guide contributeur

Direction de recherche
Examinez la fonction im2col nhwc personnalisée existante et la fonction im2col intégrée d'ORT. Ensuite, refactorez en déplaçant l'appel im2col à l'intérieur de la boucle for qui itère sur les groupes, afin qu'il opère sur un groupe à la fois.
Stack technique
cpp
Domaine
backendapi
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
C++ONNXRuntime internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60