scalanlp/breeze

csvread/csvwrite separator

Open

#357 ouverte le 30 janv. 2015

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)Scala (690 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (3 453 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

I only started using breeze a few hours ago so I may be wrong on this.

I tried to load a space separated file but I could only specify one character as a separator. Unfortunately, my dataset's separator is not consistent(2-4 spaces) as are many other datasets. Wouldn't it be better if the separator is a regular expression? That way I can specify separators like: csvread(file, "\\s+") or csvread(file, ",\\s?")

With regards to csvwrite, I may want to use a comma followed by a space.

Cheers

Guide contributeur

Direction de recherche
Localisez les fonctions csvread et csvwrite dans la bibliothèque Breeze, probablement dans le package `breeze.linalg.csv` ou similaire. Examinez comment le paramètre séparateur est actuellement utilisé (caractère unique) et modifiez la signature pour accepter une chaîne ou un motif regex. Mettez à jour la logique d'analyse pour diviser les lignes en utilisant la regex. Considérez la rétrocompatibilité en autorisant les entrées de caractère et regex. Vérifiez les tests et exemples existants pour garantir l'exactitude.
Stack technique
scala
Domaine
databackend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
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.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Understanding of ScalaFamiliarity with regex
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50