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

ENGINE機能移植にあたってのライセンス周り #874

Open
3 tasks done
qryxip opened this issue Nov 17, 2024 · 10 comments
Open
3 tasks done

ENGINE機能移植にあたってのライセンス周り #874

qryxip opened this issue Nov 17, 2024 · 10 comments

Comments

@qryxip
Copy link
Member

qryxip commented Nov 17, 2024

不具合の内容

VOICEVOX/voicevox_engine#1085 (comment)の続きです。

COREにENGINEの機能を実装するにあたり、ライセンスが問題となっています。これについて @Hiroshiba さんと話したところ、"hihoライセンス"でどうにかできるという見解が得られました。

ただ私としては以前たれぱんさんがおっしゃってた懸念にも同意できますし、できればMITでライセンスできるよう許諾を取った上で進めていきたいなと思っています。連絡が取れなかった場合でも、co-authorにはした上で事後でも通知はした方がよいかなと。

そこでちょっとコミットのルールを定めたいかなと思います。ENGINEからコードを転用してきた部分のコミットすべてに、次のものを織り込むようにします。

  1. Co-authored-by(これにより当人に通知が飛ぶはず)
  2. 連絡が取れた場合のみ、許諾が確認できるコメントへのリンク
  3. 連絡が取れた場合のみ、毎PRごとに更に許諾を取りSwitch binstalk-downloader license to "Apache-2.0 OR MIT" cargo-bins/cargo-binstall#542のようなSigned-off-by
  4. 連絡が取れなかった場合のみ、"hihoライセンス"経由でコードを取り込む旨と、Cc: @ユーザーネーム

とりあえずは私一人がこのルールで運用し、身動きが取り辛くならないかの確認をしたいと思ってます。

現象・ログ

再現手順

期待動作

VOICEVOXのバージョン

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

@qryxip
Copy link
Member Author

qryxip commented Nov 17, 2024

@y-chan (Yuto Ashida <[email protected]>)
@tarepan (tarepan <[email protected]>)
@rokujyushi (黒猫大福 <[email protected]>)
@X-20A (X-20A <[email protected]>)
@sabonerune (sabonerune <[email protected]>)

ご無沙汰しております。皆さんがVOICEVOX ENGINEで書いたコードを、MITライセンスとしてVOICEVOX COREに転用することへの許諾を頂けないでしょうか?経緯については上記の通りとなります。

@sabonerune
Copy link
Contributor

@qryxip 自分はVOICEVOX COREへの転用しても問題ありません。

@y-chan
Copy link
Member

y-chan commented Nov 18, 2024

私も問題ないです!Coreがより便利になればと...!

@rokujyushi
Copy link

私は問題ないです。

qryxip added a commit to qryxip/voicevox_core that referenced this issue Nov 19, 2024
VOICEVOX/voicevox_engine#1308VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

COREへのENGINE機能移植の一環として、 @rokujyushi さんと @sabonerune
さんの許諾のもと、MITライセンスとしてライセンスする。

Co-Authored-By: 黒猫大福 <[email protected]>
Co-Authored-By: sabonerune <[email protected]>
Refs: VOICEVOX#874 (comment)
Refs: VOICEVOX#874 (comment)
qryxip added a commit to qryxip/voicevox_core that referenced this issue Nov 20, 2024
VOICEVOX/voicevox_engine#1308VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

TODO: @X-20A さんの許諾を取るか、"hihoライセンス"経由で取り込む旨を書く

Co-Authored-By: VOICEVOX#874 (comment)
Co-Authored-By: sabonerune <[email protected]>
Refs: VOICEVOX#874 (comment)
qryxip added a commit to qryxip/voicevox_core that referenced this issue Nov 20, 2024
VOICEVOX/voicevox_engine#1308VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

TODO: @X-20A さんの許諾を取るか、"hihoライセンス"経由で取り込む旨を書く

Co-Authored-By: X-20A <[email protected]>
Co-Authored-By: sabonerune <[email protected]>
Refs: VOICEVOX#874 (comment)
@tarepan
Copy link
Contributor

tarepan commented Nov 20, 2024

私も問題ないです

@X-20A
Copy link
Contributor

X-20A commented Nov 20, 2024

私は問題ありません

qryxip added a commit to qryxip/voicevox_core that referenced this issue Nov 21, 2024
VOICEVOX/voicevox_engine#1308VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

@Hiroshiba さんと以下の2名の許諾のもと、 VOICEVOX#874 にのっとりMITライセンスと
してライセンスする。

* @X-20A (VOICEVOX/voicevox_engine#1308)
* @sabonerune (VOICEVOX/voicevox_engine#1425)

Co-Authored-By: X-20A <[email protected]>
Co-Authored-By: sabonerune <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Refs: VOICEVOX#874 (comment)
Refs: VOICEVOX#874 (comment)
@qryxip
Copy link
Member Author

qryxip commented Nov 21, 2024

ありがとうございました! 🙇

コミットメッセージの内容ですが、次のようにしたいと思います。

feat: `pause_length{,_scale}`をデフォルト値限定で受け入れる

VOICEVOX/voicevox_engine#1308 と VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

@Hiroshiba さんと以下の2名の許諾のもと、 #874 にのっとりMITライセンスと
してライセンスする。

* @X-20A (VOICEVOX/voicevox_engine#1308)
* @sabonerune (VOICEVOX/voicevox_engine#1425)

Co-Authored-By: X-20A <[email protected]>
Co-Authored-By: sabonerune <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Refs: https://github.com/VOICEVOX/voicevox_core/issues/874#issuecomment-2489461101
Refs: https://github.com/VOICEVOX/voicevox_core/issues/874#issuecomment-2482637392

qryxip added a commit that referenced this issue Nov 23, 2024
feat: `pause_length{,_scale}`をデフォルト値限定で受け入れる

`AudioQuery`に`pause_length{,_scale}`を追加する。ただしそれぞれ`null`と
`1.`のみを許し、無音時間調整自体はまだ実装しない。

VOICEVOX/voicevox_engine#1308VOICEVOX/voicevox_engine#1425 の一部
を参考にコードを書いた。

@Hiroshiba さんと以下の2名の許諾のもと、 #874 にのっとりMITライセンスと
してライセンスする。

* @X-20A (VOICEVOX/voicevox_engine#1308)
* @sabonerune (VOICEVOX/voicevox_engine#1425)

Co-authored-by: X-20A <[email protected]>
Co-authored-by: sabonerune <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Refs: VOICEVOX/voicevox_engine#1308, VOICEVOX/voicevox_engine#1425
Refs: #874 (comment)
Refs: #874 (comment)
@qryxip
Copy link
Member Author

qryxip commented Dec 21, 2024

@Segu-g すみません、ちょっと #900 ということをやりたく、MITライセンスとしてVOICEVOX COREに転用することへの許諾を頂けないでしょうか?

@Segu-g
Copy link
Member

Segu-g commented Dec 22, 2024

了解致しました。問題ありません。

@qryxip
Copy link
Member Author

qryxip commented Dec 22, 2024

ありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants