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

Aberta
#474 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
52/100
Tipo de issue
Refatoração
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
python
Domínio
data

Direção de pesquisa

Comece em wfdb.processing.peaks.find_local_peaks e inspecione a ordem dos loops e a condição de saída mostradas na issue. Confirme que o loop final é inalcançável após o loop anterior, remova apenas esse loop morto e verifique se o comportamento existente da função permanece inalterado.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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
Linguagem predominante
Jupyter Notebook
Estrelas
853
Forks
322
Nenhum PR com merge em 30d

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de MIT-LCP/wfdb-python

Todas as issues de MIT-LCP/wfdb-python

Issues semelhantes

Mais issues de Data Engineering

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.