Change approach for allowing `seg_name` and `seg_len` to be lists
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 48/100
Piste de recherche
Commencez par examiner LIST_FIELDS dans wfdb/io/record.py autour de la ligne 1675, ainsi que la gestion actuelle de SEGMENT_SPECS. La modification sera terminée lorsque seg_name et seg_len seront gérés via LIST_FIELDS et que l’écriture d’un MultiSegmentRecord continuera de les prendre en charge.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
In https://github.com/MIT-LCP/wfdb-python/pull/514 an update was made to allow seg_name and seg_len to be lists. This is needed for writing a MultiSegmentRecord.
@bemoody , suggested an alternate approach:
But I think what we really want is, rather than changing the types in SEGMENT_SPECS, we want to add "seg_name" and "seg_len" to LIST_FIELDS (wfdb/io/record.py, line 1675.)
This issue is being opened to track implementing this alternate approach.
- Langage dominant
- Jupyter Notebook
- Étoiles
- 853
- Forks
- 323
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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 MIT-LCP/wfdb-python
-
`find_local_peaks` ignores the sample at `i + radius`, so points on a rising slope are returned as peaksPeut-être pris @rmz-oz l’a pris il y a 9 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
MIT-LCP/wfdb-python#572 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
MIT-LCP/wfdb-python#568 ·
-
Add validation of segments when writing a `MultiRecord`Peut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 48/100
MIT-LCP/wfdb-python#557 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 58/100
MIT-LCP/wfdb-python#554 ·
-
WFDB path ignoredOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
MIT-LCP/wfdb-python#545 ·
Toutes les issues de MIT-LCP/wfdb-python
Issues similaires
-
[Export] Add Column Data Type Inferrer for CSV Backup Imports in src/lib/export/type-inferrer.tsOuvertebeginner friendly community contributions-welcome enhancement good first issue hacktoberfest help wanted up-for-grabs
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
from_sktime_dataset fails or reorders rows for non-default integer indexesPeut-être pris @ktz03 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
tslearn-team/tslearn#773 ·
Les mainteneurs répondent en général sous 1 jour
-
request-theme
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
LizardByte/ThemerrDB#8910 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
transitmatters/mbta-performance#104 ·
Les mainteneurs répondent en général sous 1 jour
-
[Anon] Imperium - Adeptus Astartes - Dark Angels: Azrael is missing the Lion Helm wargear abilityOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100