Errors in `get_record_list`
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Jupyter Notebook
- Estrellas
- 853
- Forks
- 322
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
import wfdb
print(wfdb.get_record_list("antimicrobial-resistance-uti")) # The 7th database from get_dbs()
Rather than returning a list of records, it returns a list of lines of a 403 Error HTML file.
['<!DOCTYPE html>', '', '<html lang="en">', ' <head>', ' <meta charset="UTF-8">', ' <title>', '403: Forbidden Access', '</title>', ... , '</html>']
Beyond either removing, renaming or fixing the dbname. I would also recommend detecting that the file accessed is an Error Page and not a valid list of records and raise a corresponding Python Error.
Also:
print(wfdb.get_record_list("bhx-brain-bounding-box")) # The 9th database from get_dbs()
Raises a Value Error:
ValueError: The database https://physionet.org/files/bhx-brain-bounding-box/1.1 has no WFDB files to download
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza en el punto de entrada get_record_list y reproduce los ejemplos antimicrobial-resistance-uti y bhx-brain-bounding-box del issue. Sigue cómo se analiza la respuesta de la base de datos y verifica después que las bases de datos válidas devuelvan nombres de registros, mientras que una página de error HTML o una base de datos sin archivos WFDB produzca un error de Python correspondiente.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- databases
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 38/100