-
Notifications
You must be signed in to change notification settings - Fork 20
[DOC]Update documents #89
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
Conversation
[FEATURE] version upgrade
[BUGFIX] fix the np.error in gensim_vec.W2V.infer_vector
[test] add test for tokenizer
Codecov Report
@@ Coverage Diff @@
## docs #89 +/- ##
===========================================
+ Coverage 99.85% 100.00% +0.14%
===========================================
Files 46 46
Lines 1364 1365 +1
===========================================
+ Hits 1362 1365 +3
+ Misses 2 0 -2
Continue to review full report at Codecov.
|
BAOOOOOM
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checked
Thanks for sending a pull request!
Please make sure you click the link above to view the contribution guidelines,
then fill out the blanks below.
Description
(Brief description on what this PR is about)
What does this implement/fix? Explain your changes.
Update some documemts.
Fix some api documents.
Add a simple flow chart.
Pull request type
Changes
Does this close any currently open issues?
#71
Any relevant logs, error output, etc?
N/A
Checklist
Before you submit a pull request, please make sure you have to following:
Essentials
Comments