Use a single `path` argument (and deprecate `pn_dir` and `file_name`)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- python
- Área
- backend-api-design
Línea de trabajo
Comienza rastreando los puntos de entrada de la API de WFDB que aceptan pn_dir y file_name; después, identifica la lógica que recupera automáticamente archivos de los servidores de PhysioNet. Define las llamadas afectadas y el comportamiento de compatibilidad antes de modificarlas. Se considera terminado cuando se admite un único argumento path, los argumentos anteriores están obsoletos y se ha eliminado la recuperación remota implícita.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently the WFDB API uses 2 arguments to specify the location of files: pn_dir and file_name. This is confusing for users and confusing for developers (e.g. see: https://github.com/MIT-LCP/wfdb-python/pull/523/files).
I think the reason for having two arguments is to support auto-detection of files (e.g. get locally if local and if not get from the PhysioNet servers). This kind of behaviour doesn't seem like a good idea. If WFDB is retrieving a remote file, it should be explicit.
My preference would be to:
- Introduce a new
pathargument - Deprecate
pn_dirandfile_name - Remove any logic that automatically retrieves files from remote servers.
- Lenguaje dominante
- Jupyter Notebook
- Estrellas
- 853
- Forks
- 322
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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.
Más de MIT-LCP/wfdb-python
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
MIT-LCP/wfdb-python#568 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
MIT-LCP/wfdb-python#557 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 58/100
MIT-LCP/wfdb-python#554 ·
-
WFDB path ignored Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
MIT-LCP/wfdb-python#545 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
MIT-LCP/wfdb-python#540 ·
Todos los issues de MIT-LCP/wfdb-python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
speaches-ai/speaches#678 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
-
[BUG] ECR GetAuthorizationToken returns a proxyEndpoint for the default region, not the request's Abiertobug ecr
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
conda-forge/spacy-feedstock#177 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100