csv2ann truncates paths containing dots, lacks write_dir, tests and docs
Mantenedores costumam responder em até 1 dia
Um pull request relacionado já foi integrado.
- #599 de @TheNetherWatcher — integrado
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 72/100
Direção de pesquisa
Start with wfdb/io/convert/csv.py, especially csv2ann near the referenced line, and compare its output path handling with csv_to_wfdb and wrann. Add tests for dotted paths and write_dir, then update docs/convert.rst to list csv2ann; done means path handling is consistent and the function is covered and documented.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Found while reviewing #591.
csv2ann in wfdb/io/convert/csv.py has some problems:
- The record name is derived with
file_name.split(".")[0](https://github.com/MIT-LCP/wfdb-python/blob/b8914e1c9c777c45571c894cb7b6579550a86f13/src/wfdb/io/convert/csv.py#L627), which breaks on paths containing dots:./x.csvbecomes""anddata/v1.2/x.csvbecomesdata/v1.os.path.splitextwould avoid this. (csv_to_wfdbalso uses the base name only, so the expected output location should be the same for both.) - There is no
write_dirparameter, unlikecsv_to_wfdbandwrann, so output paths depend on the input path and the current directory. - It has no tests.
- It is not included in the API docs (
docs/convert.rstonly listscsv_to_wfdb).
- Linguagem predominante
- Jupyter Notebook
- Estrelas
- 853
- Forks
- 324
- Merge médio
- 15h
- PRs com merge (30d)
- 32
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de MIT-LCP/wfdb-python
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
MIT-LCP/wfdb-python#618 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 72/100
MIT-LCP/wfdb-python#590 ·
Mantenedores costumam responder em até 1 dia
-
pandas 3.0: three remaining set_index(<str column>.values) calls still raise TypeError (custom-labels paths)Talvez já em andamento @wenh06 assumiu há 8 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
MIT-LCP/wfdb-python#574 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
`find_local_peaks` ignores the sample at `i + radius`, so points on a rising slope are returned as peaksTalvez já em andamento @rmz-oz assumiu há 29 dias. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
MIT-LCP/wfdb-python#572 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
MIT-LCP/wfdb-python#617 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de MIT-LCP/wfdb-python
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
Slaymish/HealthAgent#7 ·
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 85/100
loopj/aiovantage#414 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
Mantenedores costumam responder em até 1 dia
-
known issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
Mantenedores costumam responder em até 1 dia
-
bug good first issue
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
CakeRepository/agentshangout#38 ·
Mantenedores costumam responder em até 1 dia