fgnt / padertorch

A collection of common functionality to simplify the design, training and evaluation of machine learning models based on pytorch with an emphasis on speech processing.
MIT License
71 stars 16 forks source link

Fix GitHub actions #147

Closed michael-kuhlmann closed 1 year ago

michael-kuhlmann commented 1 year ago

GitHub actions currently do not pass because pb_bss cannot be successfully installed due to an old version of setuptools that is used. This pull request fixes this and upgrades the action versions of checkout and setup-python.

michael-kuhlmann commented 1 year ago

If you do not like to have the contrib commits in this PR, I can first merge the contrib changes and then create a new clean PR for this issue.

boeddeker commented 1 year ago

If you do not like to have the contrib commits in this PR, I can first merge the contrib changes and then create a new clean PR for this issue.

We can use one PR to merge both. It is unlikely, that we have second PR, that should be merged before your praat-textgrids code.

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (2a4c391) 77.12% compared to head (dc9f016) 77.12%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #147 +/- ## ======================================= Coverage 77.12% 77.12% ======================================= Files 46 46 Lines 3729 3734 +5 ======================================= + Hits 2876 2880 +4 - Misses 853 854 +1 ``` | [Impacted Files](https://app.codecov.io/gh/fgnt/padertorch/pull/147?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [padertorch/modules/dual\_path\_rnn.py](https://app.codecov.io/gh/fgnt/padertorch/pull/147?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-cGFkZXJ0b3JjaC9tb2R1bGVzL2R1YWxfcGF0aF9ybm4ucHk=) | `68.34% <ø> (ø)` | | | [padertorch/configurable.py](https://app.codecov.io/gh/fgnt/padertorch/pull/147?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#diff-cGFkZXJ0b3JjaC9jb25maWd1cmFibGUucHk=) | `79.82% <100.00%> (+0.03%)` | :arrow_up: | ... and [4 files with indirect coverage changes](https://app.codecov.io/gh/fgnt/padertorch/pull/147/indirect-changes?src=pr&el=tree-more&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.

michael-kuhlmann commented 1 year ago

I have added python 3.9 and 3.10 and adapted two doctests to work with the latest versions of torch and python:

boeddeker commented 1 year ago

LGTM. The azure tests can be ignored, they got somehow canceled.