rapidsai/cuml

[FEA] cpp-bench: refactor the `read_csv` method into a more common place

Open

#1.155 aberto em 27 de set. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Python (644 forks)auto 404
? - Needs TriageCUDA / C++Tech Debtfeature requestgood first issuequick-hit

Métricas do repositório

Stars
 (5.223 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Filing this issue as per the comment by @cjnolet here. It certainly makes sense to move this logic somewhere which can help increase reuse. However, I'd also like to caution that this function was tailor-made for numerical values only and also only tested for dense csv files! So, we might want to spend some effort on making this function slightly more generic, before migrating it for a wider use inside cuML.

Guia do colaborador

Direção de pesquisa
Inspecione o método `read csv` atual no código do cpp bench, entenda suas dependências e uso, e identifique um local comum adequado (por exemplo, biblioteca de utilitários) dentro do cuML para refatorá lo para reutilização entre componentes C++.
Pilha de tecnologia
cpppython
Domain
backendmachine learning
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
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.
Principalmente claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40