Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[breaking] Update submodules #109

Merged
merged 8 commits into from
May 28, 2024
Merged

Conversation

yamachu
Copy link
Owner

@yamachu yamachu commented May 27, 2024

コード上の変更

  • ErrorCodeが足された
  • ModelIdの型が変わった

breaking

  • OnnxRuntimeのアップデート
    • 1.14.0 -> 1.17.3
  • voicevox_coreのsample.vvmの提供方法が変わったことによるCIの方法の変更
    • おそらく指定バージョンのテストは壊れてる

@yamachu yamachu added the breaking Breaking changes label May 27, 2024
@yamachu yamachu changed the title Update submodules [breaking] Update submodules May 27, 2024
@yamachu
Copy link
Owner Author

yamachu commented May 27, 2024

/mq

とは言えmainブランチを追い続けるのがこのプロジェクトなので、mergeする

Copy link
Contributor

Copy link
Contributor

Testing: https://github.com/yamachu/VoicevoxCoreSharp/actions/runs/9254883152
You can view state by GitHub CLI below command:

  • gh run view --repo yamachu/VoicevoxCoreSharp 9254883152
  • gh run watch --repo yamachu/VoicevoxCoreSharp 9254883152

If you want to cancel the test, you can cancel it by GitHub CLI below command:

  • gh run cancel --repo yamachu/VoicevoxCoreSharp 9254883152

@@ -50,7 +50,8 @@ jobs:
- run: |
cp open_jtalk_dic_utf_8-1.11.tar.gz ./tests/VoicevoxCoreSharp.Core.Tests/resources
tar zxf ./tests/VoicevoxCoreSharp.Core.Tests/resources/open_jtalk_dic_utf_8-1.11.tar.gz -C tests/VoicevoxCoreSharp.Core.Tests/resources/
- run: cp -r binding/voicevox_core/model ./tests/VoicevoxCoreSharp.Core.Tests/resources
- name: sample.vvmを自前でzipコマンドで作成する
run: mkdir -p tests/VoicevoxCoreSharp.Core.Tests/resources/model; cd binding/voicevox_core/model/sample.vvm; zip -r ../../../../tests/VoicevoxCoreSharp.Core.Tests/resources/model/sample.vvm .
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

本来は crates/test_util/data/model 以下に生成された sample.vvm を使用するのがいいが、cacheなどの関係上変更が若干面倒なので、やるとしても別のPRで出来るようにしたい

@yamachu
Copy link
Owner Author

yamachu commented May 27, 2024

/mq

Copy link
Contributor

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

8 similar comments
@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

@yamachu
Copy link
Owner Author

yamachu commented May 28, 2024

/mq

Copy link
Contributor

Copy link
Contributor

Testing: https://github.com/yamachu/VoicevoxCoreSharp/actions/runs/9269959809
You can view state by GitHub CLI below command:

  • gh run view --repo yamachu/VoicevoxCoreSharp 9269959809
  • gh run watch --repo yamachu/VoicevoxCoreSharp 9269959809

If you want to cancel the test, you can cancel it by GitHub CLI below command:

  • gh run cancel --repo yamachu/VoicevoxCoreSharp 9269959809

@actions-commit-pr-dekiru-kun actions-commit-pr-dekiru-kun bot merged commit 08d6776 into main May 28, 2024
4 checks passed
@actions-commit-pr-dekiru-kun actions-commit-pr-dekiru-kun bot deleted the update-submodules branch May 28, 2024 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant