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.

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

compare函数的问题 #60

Closed
jreros opened this issue Apr 26, 2018 · 3 comments
Closed

compare函数的问题 #60

jreros opened this issue Apr 26, 2018 · 3 comments

Comments

@jreros
Copy link

jreros commented Apr 26, 2018

调用compare函数时，交换sen1和sen2的值，得到差异很大的结果，例如：

synonyms.compare("教学", "老师")
0.879
synonyms.compare("老师", "教学")
0.194
hailiang-wang referenced this issue Apr 28, 2018
@huyingxi
Update synonyms.py
c580b3d 
update _nearby_levenshtein_distance
which is calculated by considering both first and second sentence's nearby words
@hailiang-wang
Copy link
Member

@jreros , @huyingxi 做了一个提交。
简单说，两个句子内部使用有顺序，现在yingxi已经优化了。
在测试通过后，会发布在下一个版本中。
Thanks.

@hailiang-wang
Copy link
Member

@huyingxi
现在你的修改，使得计算量增加不少。有更简单的方案。

hailiang-wang added a commit that referenced this issue May 28, 2018
@hailiang-wang
#60 compare 支持交换句子
4a44eff
@hailiang-wang
Copy link
Member

Fixed in v3.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jreros @hailiang-wang