-
Notifications
You must be signed in to change notification settings - Fork 42
compute_and_store_features() got an unexpected keyword argument 'augment_fn' #22
Comments
what are the |
Maybe it was a change in lhotse interface and we need to change our code.
…On Wed, Nov 18, 2020 at 6:30 PM Haowen Qiu ***@***.***> wrote:
what are the dev-clean-3 and train-clean-5 here? can you try to run with
the original script to see if you get the same error? I mean, just clone
the repo and don't change any code in it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#22 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAZFLO5AHMNHY27LQY5MW7DSQOO5XANCNFSM4TZXTWQQ>
.
|
Not sure, will check |
OK, it should be
but in master branch of lhotse, it now is
@pzelasko can you update pip package of lhotse? @lvchigo for now, you can just do
to install lhotse from local src (don't forget uninstall your previous version) |
OK I’ll release a new Lhotse version. Was waiting to accumulate some changes first. BTW Did you guys figure out how to auto-upload nightly builds to pip?
… Wiadomość napisana przez Haowen Qiu ***@***.***> w dniu 11/18/20, o godz. 06:38:
OK, it should be augmenter with pip package lhotse 0.1.1,
CutSet.compute_and_store_features.__code__.co_varnames
('self', 'extractor', 'storage', 'augmenter', 'executor', 'mix_eagerly', 'futures', 'cut', 'cut_set')
but in master branch of lhotse, it now is augment_fn
>>> lhotse.CutSet.compute_and_store_features.__code__.co_varnames
('self', 'extractor', 'storage', 'augment_fn', 'executor', 'mix_eagerly', 'futures', 'cut', 'cut_set')
https://github.com/lhotse-speech/lhotse/blob/2167c774383b815dc0929e9cc6bb7d26eec0fccb/lhotse/cut.py#L325
@pzelasko can you update pip package of lhotse?
@lvchigo for now, you can just do
git clone https://github.com/lhotse-speech/lhotse
cd lhotse
pip install -e '.[dev]'
to install lhotse from local src (don't forget uninstall your previous version)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
I had something that might be usefulset up here:
https://github.com/sequitur-g2p/sequitur-g2p/blob/master/.travis.yml
(it's not automatic daily, it needs a tag, but you could tag automatically,
I guess)
y.
…On Wed, Nov 18, 2020 at 7:50 AM Piotr Żelasko ***@***.***> wrote:
OK I’ll release a new Lhotse version. Was waiting to accumulate some
changes first. BTW Did you guys figure out how to auto-upload nightly
builds to pip?
> Wiadomość napisana przez Haowen Qiu ***@***.***> w dniu
11/18/20, o godz. 06:38:
>
>
> OK, it should be augmenter with pip package lhotse 0.1.1,
>
> CutSet.compute_and_store_features.__code__.co_varnames
> ('self', 'extractor', 'storage', 'augmenter', 'executor', 'mix_eagerly',
'futures', 'cut', 'cut_set')
> but in master branch of lhotse, it now is augment_fn
>
> >>> lhotse.CutSet.compute_and_store_features.__code__.co_varnames
> ('self', 'extractor', 'storage', 'augment_fn', 'executor',
'mix_eagerly', 'futures', 'cut', 'cut_set')
>
>
https://github.com/lhotse-speech/lhotse/blob/2167c774383b815dc0929e9cc6bb7d26eec0fccb/lhotse/cut.py#L325
>
> @pzelasko can you update pip package of lhotse?
>
> @lvchigo for now, you can just do
>
> git clone https://github.com/lhotse-speech/lhotse
> cd lhotse
> pip install -e '.[dev]'
> to install lhotse from local src (don't forget uninstall your previous
version)
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or unsubscribe.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#22 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACUKYX74PGF6WBA7RF4F3U3SQO7H5ANCNFSM4TZXTWQQ>
.
|
|
RE automatic builds that upload to pip, I think @csukuangfj might be able to comment. |
GitHub actions support uploading wheels to PyPI. As for the password, it can be saved in the settings of your account and be accessed in GitHub actions as environment variables. Others can only see the name of the environment variables, but not their values. So the username and password can still be kept as secrets.
GitHub actions can be triggered by some events, e.g., on the creation of a tag, on a new pull request, on a merge into a specific branch, at some specified time points per day etc.
Sent from myMail for iOS
Thursday, 19 November 2020, 11:20 +0800 from notifications@github.com <notifications@github.com>:
…RE automatic builds that upload to pip, I think @csukuangfj might be able to comment.
I believe there we have some kind of automated build but currently only uploads artifacts to github, not pip which would require password.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub , or unsubscribe .
|
Thanks! I’ll check it out at some point and set it up.
… Wiadomość napisana przez Fangjun Kuang ***@***.***> w dniu 11/18/20, o godz. 22:45:
GitHub actions support uploading wheels to PyPI. As for the password, it can be saved in the settings of your account and be accessed in GitHub actions as environment variables. Others can only see the name of the environment variables, but not their values. So the username and password can still be kept as secrets.
GitHub actions can be triggered by some events, e.g., on the creation of a tag, on a new pull request, on a merge into a specific branch, at some specified time points per day etc.
Sent from myMail for iOS
Thursday, 19 November 2020, 11:20 +0800 from ***@***.*** ***@***.***>:
>RE automatic builds that upload to pip, I think @csukuangfj might be able to comment.
>I believe there we have some kind of automated build but currently only uploads artifacts to github, not pip which would require password.
>—
>You are receiving this because you were mentioned.
>Reply to this email directly, view it on GitHub , or unsubscribe .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
pip list:
python 3.7
torch 1.6.0+cu101
torchaudio 0.6.0
torchvision 0.7.0+cu101
lhotse 0.1.1
k2 0.1
The text was updated successfully, but these errors were encountered: