Third loop in wfdb.processing.peaks.find_local_peaks will not be executed and can be removed

Ouverte
#474 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
1/5
Temps estimé
Moins d'une heure
Accessibilité débutants
52/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
python
Domaine
data

Piste de recherche

Commencez dans wfdb.processing.peaks.find_local_peaks et examinez l’ordre des boucles ainsi que la condition de sortie présentés dans l’issue. Confirmez que la dernière boucle est inaccessible après la boucle précédente, supprimez uniquement cette boucle morte et vérifiez que le comportement existant de la fonction reste inchangé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

In find_local_peaks() in wfdb.processing.peaks, the last of three loops will not be execute because the condition i < len(sig) is already false when the second loop exits:
[first loop omitted]

while i < len(sig):
    if sig[i] == max(sig[i - radius : i + radius]):
        peak_inds.append(i)
        i += radius
    else:
        i += 1

while i < len(sig):
    if sig[i] == max(sig[i - radius :]):
        peak_inds.append(i)
        i += radius
    else:
        i += 1
Langage dominant
Jupyter Notebook
Étoiles
853
Forks
322
Aucune PR mergée en 30 j

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de MIT-LCP/wfdb-python

Toutes les issues de MIT-LCP/wfdb-python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.