ucb-bar/onnxruntime-riscv

Refactoring: Use the recently added Relu int8 type

Aberta

#25 aberto em 27 de jan. de 2021

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (0 responsável)C++ (37 forks)auto 404
enhancementgood first issue

Métricas do repositório

Stars
 (93 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Estude o PR upstream (onnx/onnx#3141) para entender o novo tipo Relu int8. Em seguida, no código do quantizador local, substitua a emissão do tipo QLinearRelu pela operação Relu padrão que agora suporta int8. Remova qualquer lógica de fusão específica para o tipo personalizado. Atualize também os testes relevantes.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70