Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add python=3.10 to CI#4627

Merged
kamo-naoyuki merged 10 commits intomasterfrom
py310
Mar 5, 2023
Merged

Add python=3.10 to CI#4627
kamo-naoyuki merged 10 commits intomasterfrom
py310

Conversation

@kamo-naoyuki
Copy link
Collaborator

No description provided.

@mergify mergify bot added CI Travis, Circle CI, etc mergify labels Sep 8, 2022
@mergify
Copy link
Contributor

mergify bot commented Feb 3, 2023

This pull request is now in conflict :(

@mergify mergify bot added the conflicts label Feb 3, 2023
@kamo-naoyuki
Copy link
Collaborator Author

@kan-bayashi Espnet depends on nnmnkwii(https://github.com/espnet/espnet/blob/master/setup.py#L67), but the requirement of nnmkkwii for numpy is not compatible with python=3.10 (https://github.com/espnet/espnet/actions/runs/4079849732/jobs/7031662783#step:6:200).

nnmkkwii is invoked only in the following script:

from nnmnkwii.io import hts

The function used here seems a simple text processing and can be implemented easily. I think nnmkkwii is too large dependency for this purpose because it includes even some cython codes.

I think rewriting it to a local function is better. How do you think?

@kan-bayashi
Copy link
Member

@r9y9 Is it possible to update nnmnkwii?

@r9y9
Copy link
Contributor

r9y9 commented Feb 3, 2023

@kan-bayashi Let me take a look.

@r9y9
Copy link
Contributor

r9y9 commented Feb 3, 2023

@kan-bayashi New release is up https://pypi.org/project/nnmnkwii/0.1.2/

@kan-bayashi
Copy link
Member

@r9y9 Thank you for blazing fast release! Super cool!

@kamo-naoyuki Could you try the new version?

@mergify mergify bot removed the conflicts label Feb 3, 2023
@kamo-naoyuki
Copy link
Collaborator Author

@r9y9 pyopenjtalk has same problem, could you have a look?

@kan-bayashi
Copy link
Member

kan-bayashi commented Feb 4, 2023

It seems Ryuichi already fixed the issue in v0.3.0 of pyopenjtalk.
I update the version in #4912

@kan-bayashi
Copy link
Member

pyopenjtalk seems to be fixed. But gensim has a problem in 3.10 :(

@mergify mergify bot added the Installation label Feb 9, 2023
@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Merging #4627 (a6b9e88) into master (435ea75) will increase coverage by 3.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4627      +/-   ##
==========================================
+ Coverage   73.48%   77.01%   +3.52%     
==========================================
  Files         606      606              
  Lines       53761    53770       +9     
==========================================
+ Hits        39508    41410    +1902     
+ Misses      14253    12360    -1893     
Flag Coverage Δ
test_integration_espnet1 66.33% <ø> (+<0.01%) ⬆️
test_integration_espnet2 47.77% <ø> (?)
test_python 66.85% <ø> (+<0.01%) ⬆️
test_utils 23.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
espnet2/slu/postencoder/conformer_postencoder.py 93.75% <0.00%> (+0.06%) ⬆️
espnet2/enh/layers/wpe.py 87.34% <0.00%> (+0.16%) ⬆️
espnet2/asr/state_spaces/s4.py 53.16% <0.00%> (+0.16%) ⬆️
espnet2/train/reporter.py 88.44% <0.00%> (+0.26%) ⬆️
espnet2/text/phoneme_tokenizer.py 75.50% <0.00%> (+0.33%) ⬆️
...net/nets/chainer_backend/deterministic_embed_id.py 68.60% <0.00%> (+0.36%) ⬆️
espnet/nets/pytorch_backend/e2e_tts_transformer.py 88.79% <0.00%> (+0.56%) ⬆️
espnet2/train/iterable_dataset.py 85.47% <0.00%> (+0.85%) ⬆️
espnet/asr/asr_utils.py 76.53% <0.00%> (+0.87%) ⬆️
espnet2/asr/state_spaces/pool.py 26.12% <0.00%> (+1.01%) ⬆️
... and 55 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kamo-naoyuki kamo-naoyuki merged commit b5b2b11 into master Mar 5, 2023
@kamo-naoyuki kamo-naoyuki deleted the py310 branch March 5, 2023 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants