vocalpy / crowsetta

A tool to work with any format for annotating vocalizations
https://crowsetta.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
49 stars 3 forks source link

BUG: Coerce GenericSeq path attributes to str #237

Closed NickleDave closed 1 year ago

NickleDave commented 1 year ago

Hoping this will fix CI failures.

We coerce the annot_path and notated_path attributes of the pandera GenericSeq schema to be strings -- apparently they get passed in as pathlib.Path on CI somehow? I can't reproduce the error locally. But coercing them to string seems to fix it. And it makes sense in general to force these attributes to be strings since there's no pathlib dtype for pandas DataFrames (without using a library like pandas-pathlib)

codecov-commenter commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (970be2f) 94.10% compared to head (c1ef7a5) 94.10%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #237 +/- ## ======================================= Coverage 94.10% 94.10% ======================================= Files 41 41 Lines 1510 1510 ======================================= Hits 1421 1421 Misses 89 89 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.