csv2ann truncates paths containing dots, lacks write_dir, tests and docs
Maintainer antworten meist innerhalb von 1 Tag
Ein zugehöriger Pull Request wurde bereits gemerged.
- #599 von @TheNetherWatcher — gemerged
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 72/100
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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).
- Vorherrschende Sprache
- Jupyter Notebook
- Sterne
- 853
- Forks
- 324
- Ø Merge
- 15 Std.
- Gemergte PRs (30 T.)
- 32
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
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
MIT-LCP/wfdb-python#618 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
MIT-LCP/wfdb-python#590 ·
Maintainer antworten meist innerhalb von 1 Tag
-
pandas 3.0: three remaining set_index(<str column>.values) calls still raise TypeError (custom-labels paths)Evtl. vergeben @wenh06 hat das vor 8 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
MIT-LCP/wfdb-python#574 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
`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 30 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
MIT-LCP/wfdb-python#572 ·
Maintainer antworten meist innerhalb von 1 Tag
-
ann2rr filters intervals by length instead of by time in the recordEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
MIT-LCP/wfdb-python#617 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in MIT-LCP/wfdb-python
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
MetricsHub/metricshub-community#1474 ·
Maintainer antworten meist innerhalb von 1 Tag
-
good first issue help wanted r-refactor
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
servo/rust-smallvec#843 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 2 Tagen
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
farion1231/cc-switch#9122 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
lbjlaq/Antigravity-Manager#3801 ·
Maintainer antworten meist innerhalb von 1 Tag