ucb-bar/onnxruntime-riscv

Refactoring: Use the recently added Relu int8 type

Aperta

#25 aperta il 27 gen 2021

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)C++ (37 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (93 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Currently we emit our own QLinearRelu type in the quantizer and do fusing based off of this. This was done to work around a previous deficiency in ONNX operators that Relu did not support int8. I recently submitted a PR to add Relu support which was recently merged in: https://github.com/onnx/onnx/pull/3141

We can now emit and use this type.

Guida contributor

Direzione di ricerca
Studia la PR upstream (onnx/onnx#3141) per comprendere il nuovo tipo Relu int8. Quindi, nel codice del quantizzatore locale, sostituisci l'emissione del tipo QLinearRelu con l'operazione Relu standard che ora supporta int8. Rimuovi qualsiasi logica di fusione specifica per il tipo personalizzato. Aggiorna anche i test pertinenti.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70