Simply pytests `subjects_csv` and `sessions_csv` by removing pandas dependency
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 42/100
Piste de recherche
Repérez la couverture pytest de subjects_csv et sessions_csv et examinez comment pandas est utilisé pour construire ou comparer les données d’exemple. Remplacez ces opérations par des Python dicts et lists, écrivez les données CSV d’exemple sous forme de chaîne multiligne et exécutez les tests pytest concernés pour confirmer qu’ils passent toujours sans la dépendance à pandas.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I think the use of pandas here is unjustified. All the operations where it's used can be just more cleanly accomplished with python datatypes: dicts and lists.
Originally posted by @dimitri-yatsenko in https://github.com/datajoint/workflow-animal/pull/9#discussion_r775087716
writing a sample CSV file with sample data can be made much simpler. Consider simply writing a multiline string into a file.
Originally posted by @dimitri-yatsenko in https://github.com/datajoint/workflow-animal/pull/9#discussion_r775087876
- Langage dominant
- Jupyter Notebook
- Étoiles
- 10
- Forks
- 44
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de datajoint/element-array-ephys
-
Migrate to DataJoint 2.x: 14 longblob attributes silently corrupt data, 1 attribute fails to declareOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 58/100
datajoint/element-array-ephys#230 ·
-
enhancement
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
datajoint/element-array-ephys#207 ·
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
datajoint/element-array-ephys#206 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
datajoint/element-array-ephys#154 · 1 commentaire ·
-
NWB-export.ipynb errorOuvertebug
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
datajoint/element-array-ephys#163 ·
Toutes les issues de datajoint/element-array-ephys
Issues similaires
-
triage
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
rjmurillo/moq.analyzers#1384 ·
-
Test LeakagePeut-être pris @garland3 l’a pris aujourd’hui. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
sandialabs/atlas-ui-3#1030 ·
Les mainteneurs répondent en général sous 1 jour
-
Offline audit truncates pytest parameter IDs containing spacesPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
dgunning/edgartools#1476 ·
Les mainteneurs répondent en général sous 1 jour
-
go-battery needs an ndsctl on PATH: TestPurchaseSessionGuardHoldsThroughTheOutcomeUnknownWindow fails on bare hosts (passes with stub)Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
OpenTollGate/tollgate-module-basic-go#726 ·
Les mainteneurs répondent en général sous 1 jour
-
phase:v3 type:harness
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour