Use a single `path` argument (and deprecate `pn_dir` and `file_name`)
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- python
- Bereich
- backend-api-design
Rechercherichtung
Beginne damit, die WFDB-API-Einstiegspunkte nachzuverfolgen, die pn_dir und file_name akzeptieren, und ermittle anschließend die Logik, die Dateien automatisch von PhysioNet-Servern abruft. Definiere die betroffenen Aufrufe und das Kompatibilitätsverhalten, bevor du sie änderst. Als abgeschlossen gilt die Arbeit, wenn ein einzelnes path-Argument unterstützt wird, die älteren Argumente als veraltet markiert sind und der implizite Abruf aus der Ferne entfernt wurde.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Jupyter Notebook
- Sterne
- 853
- Forks
- 323
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus MIT-LCP/wfdb-python
-
`find_local_peaks` ignores the sample at `i + radius`, so points on a rising slope are returned as peaksEvtl. vergeben @rmz-oz hat das vor 10 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
MIT-LCP/wfdb-python#572 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
MIT-LCP/wfdb-python#568 ·
-
Add validation of segments when writing a `MultiRecord`Evtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100
MIT-LCP/wfdb-python#557 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 58/100
MIT-LCP/wfdb-python#554 ·
-
WFDB path ignoredOffen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100
MIT-LCP/wfdb-python#545 ·
Alle Issues in MIT-LCP/wfdb-python
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
maxrave-dev/SimpMusic#2649 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
product/backlog
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
TencentBlueKing/bk-nodemgr#3645 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 2 Tagen
-
Objects.List treats versions=false (and similar flags) as trueEvtl. vergeben @andaaron hat das heute übernommen. Offenpriority: p2 type: bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
googleapis/storage-testbench#829 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
[Bug]: memory provider treats 1:1 DMs as shared surfaces, so DM turns never receive reviewed recordsOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 79/100
rlaope/oh-my-hermes#2088 ·
Maintainer antworten meist innerhalb von 1 Tag