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.

Sign up for GitHub

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

Jump to bottom

[sentencepiece] Fixes #1491, fix NPE bug #1492

Merged

frankfliu merged 1 commit into deepjavalibrary:master from frankfliu:sp

Feb 3, 2022

Contributor

frankfliu commented Feb 3, 2022 •

edited

Loading

Change-Id: Ib86efb38def0bcb21862c5493683395071eb3592

Description

Fixes #1491

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here


          [sentencepiece] Fixes deepjavalibrary#1149, fix NPE bug

c09baa0

Change-Id: Ib86efb38def0bcb21862c5493683395071eb3592

frankfliu requested a review from zachgk as a code owner

February 3, 2022 19:20

frankfliu changed the title ~~[sentencepiece] Fixes #1149, fix NPE bug~~ [sentencepiece] Fixes #1491, fix NPE bug

zachgk approved these changes

View reviewed changes

frankfliu merged commit 339bc87 into deepjavalibrary:master

frankfliu deleted the sp branch

February 3, 2022 20:38

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

Labels

None yet