search

CPJKU / partitura

A python package for handling modern staff notation of music
https://partitura.readthedocs.io
Apache License 2.0
243 stars 18 forks source link

removes torch dependency #255

Closed sildater closed 1 year ago

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: +0.07 :tada:

Comparison is base (ff21c9a) 85.46% compared to head (bfeb225) 85.54%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #255 +/- ## =========================================== + Coverage 85.46% 85.54% +0.07% =========================================== Files 74 74 Lines 12824 12810 -14 =========================================== - Hits 10960 10958 -2 + Misses 1864 1852 -12 ``` | [Impacted Files](https://app.codecov.io/gh/CPJKU/partitura/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CPJKU) | Coverage Δ | | |---|---|---| | [partitura/musicanalysis/performance\_codec.py](https://app.codecov.io/gh/CPJKU/partitura/pull/255?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=CPJKU#diff-cGFydGl0dXJhL211c2ljYW5hbHlzaXMvcGVyZm9ybWFuY2VfY29kZWMucHk=) | `87.03% <0.00%> (+3.58%)` | :arrow_up: |

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

CarlosCancino-Chacon commented 1 year ago

This pull request removes torch as an additional dependency (since it was only used optionally in two functions, and removing the dependency does not change the behavior of the package).