-
Notifications
You must be signed in to change notification settings - Fork 41
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
License Issues #1
Comments
Hi @singletongue! Many thanks for your group sharing the code for bert-japanese. Indeed we have adopted part of your Wikipedia dataset construction code, and I am sincerely sorry for not conforming to the same license. We will take action soon and update our codebase with an easy-to-see update message. Best |
Thank you for your swift response, @ZHAOTING. Best regards |
@singletongue -san, in order to comply with the Apache 2.0 redistribution requirements, we have to 1) keep the original license claim in the adopted file/code, and 2) explicitly note the modification. Therefore, would you please add a short license claim comment (like this ) at the beginning of your make_corpus_wiki.py file, so that we are able to use the exact same claim in our modified file? |
Thank you for your response, @ZHAOTING. I have prepended the license claims to the codes. |
@singletongue Our license has been updated. And an update log has been added to README. Thank you! |
I have checked the license and README and confirmed that everything should be fine. Thank you! |
Hi, I'm @singletongue, a maintainer of the cl-tohoku/bert-japanese.
Thank you for sharing your great work.
However, I'm a little concerned that some parts of your code
src/corpus/build_pretrain_dataset.py
are possibly taken from our codemake_corpus_wiki.py
.Since we are releasing our codes under the Apache License 2.0, It might be better if you adopted the same license, not the MIT license.
Thank you.
The text was updated successfully, but these errors were encountered: