Fix bugs in Paddle seg and Paddle postag (fxsjy#789)

* fix bugs in paddle seg and paddle postag * fix compat in checking paddle
VitoCheung · Dec 24, 2019 · 2791009 · 2791009
1 parent 9dc8e6d
commit 2791009
Show file tree

Hide file tree

Showing 4 changed files with 111 additions and 3 deletions.
diff --git a/jieba/__init__.py b/jieba/__init__.py
@@ -289,6 +289,9 @@ def cut(self, sentence, cut_all = False, HMM = True,use_paddle = False):
             is_paddle_installed = check_paddle_install()
         sentence = strdecode(sentence)
         if use_paddle == True and is_paddle_installed == True and import_paddle_check == True:
+            if sentence is None or sentence == "" or sentence == u"":
+                yield sentence
+                return
             import jieba.lac_small.predict as predict
             results = predict.get_sent(sentence)
             for sent in results:

diff --git a/jieba/_compat.py b/jieba/_compat.py
@@ -25,7 +25,7 @@ def import_paddle():
     import_paddle_check = False
     try:
         import paddle
-        if paddle.__version__ == '1.6.1':
+        if paddle.__version__ >= '1.6.1' or paddle.__version__ >= u'1.6.1':
             import paddle.fluid as fluid
             import jieba.lac_small.predict as predict
         import_paddle_check = True
@@ -75,9 +75,9 @@ def check_paddle_install():
     is_paddle_installed =  False
     try:
         import paddle
-        if imp.find_module('paddle') and paddle.__version__ == '1.6.1':
+        if imp.find_module('paddle') and (paddle.__version__ >= '1.6.1' or paddle.__version__ >= u'1.6.1'):
             is_paddle_installed = True
-        elif paddle.__version__ != '1.6.1':
+        elif paddle.__version__ < '1.6.1':
             is_paddle_installed = False
             default_logger.debug("Check the paddle version is not correct, please\
             use command to install paddle: pip uninstall paddlepaddle(-gpu), \

diff --git a/jieba/posseg/__init__.py b/jieba/posseg/__init__.py
@@ -281,6 +281,10 @@ def cut(sentence, HMM=True, use_paddle=False):
         import_paddle_check = import_paddle()
         is_paddle_installed = check_paddle_install()
     if use_paddle==True and is_paddle_installed == True and import_paddle_check == True:
+        if sentence is None or sentence == "" or sentence == u"":
+            yield pair(None, None)
+            return
+        import jieba.lac_small.predict as predict
         sents,tags = predict.get_result(strdecode(sentence))
         for i,sent in enumerate(sents):
             if sent is None or tags[i] is None:

diff --git a/test/test_paddle_postag.py b/test/test_paddle_postag.py
@@ -0,0 +1,101 @@
+#encoding=utf-8
+import sys
+sys.path.append("../")
+import jieba.posseg as pseg
+
+
+def cuttest(test_sent):
+    result = pseg.cut(test_sent, use_paddle=True)
+    for word, flag in result:
+        print('%s %s' % (word, flag))
+
+
+if __name__ == "__main__":
+    cuttest("这是一个伸手不见五指的黑夜。我叫孙悟空，我爱北京，我爱Python和C++。")
+    cuttest("我不喜欢日本和服。")
+    cuttest("雷猴回归人间。")
+    cuttest("工信处女干事每月经过下属科室都要亲口交代24口交换机等技术性器件的安装工作")
+    cuttest("我需要廉租房")
+    cuttest("永和服装饰品有限公司")
+    cuttest("我爱北京天安门")
+    cuttest("abc")
+    cuttest("隐马尔可夫")
+    cuttest("雷猴是个好网站")
+    cuttest("“Microsoft”一词由“MICROcomputer（微型计算机）”和“SOFTware（软件）”两部分组成")
+    cuttest("草泥马和欺实马是今年的流行词汇")
+    cuttest("伊藤洋华堂总府店")
+    cuttest("中国科学院计算技术研究所")
+    cuttest("罗密欧与朱丽叶")
+    cuttest("我购买了道具和服装")
+    cuttest("PS: 我觉得开源有一个好处，就是能够敦促自己不断改进，避免敞帚自珍")
+    cuttest("湖北省石首市")
+    cuttest("湖北省十堰市")
+    cuttest("总经理完成了这件事情")
+    cuttest("电脑修好了")
+    cuttest("做好了这件事情就一了百了了")
+    cuttest("人们审美的观点是不同的")
+    cuttest("我们买了一个美的空调")
+    cuttest("线程初始化时我们要注意")
+    cuttest("一个分子是由好多原子组织成的")
+    cuttest("祝你马到功成")
+    cuttest("他掉进了无底洞里")
+    cuttest("中国的首都是北京")
+    cuttest("孙君意")
+    cuttest("外交部发言人马朝旭")
+    cuttest("领导人会议和第四届东亚峰会")
+    cuttest("在过去的这五年")
+    cuttest("还需要很长的路要走")
+    cuttest("60周年首都阅兵")
+    cuttest("你好人们审美的观点是不同的")
+    cuttest("买水果然后来世博园")
+    cuttest("买水果然后去世博园")
+    cuttest("但是后来我才知道你是对的")
+    cuttest("存在即合理")
+    cuttest("的的的的的在的的的的就以和和和")
+    cuttest("I love你，不以为耻，反以为rong")
+    cuttest("因")
+    cuttest("")
+    cuttest("hello你好人们审美的观点是不同的")
+    cuttest("很好但主要是基于网页形式")
+    cuttest("hello你好人们审美的观点是不同的")
+    cuttest("为什么我不能拥有想要的生活")
+    cuttest("后来我才")
+    cuttest("此次来中国是为了")
+    cuttest("使用了它就可以解决一些问题")
+    cuttest(",使用了它就可以解决一些问题")
+    cuttest("其实使用了它就可以解决一些问题")
+    cuttest("好人使用了它就可以解决一些问题")
+    cuttest("是因为和国家")
+    cuttest("老年搜索还支持")
+    cuttest("干脆就把那部蒙人的闲法给废了拉倒！RT @laoshipukong : 27日，全国人大常委会第三次审议侵权责任法草案，删除了有关医疗损害责任“举证倒置”的规定。在医患纠纷中本已处于弱势地位的消费者由此将陷入万劫不复的境地。 ")
+    cuttest("大")
+    cuttest("")
+    cuttest("他说的确实在理")
+    cuttest("长春市长春节讲话")
+    cuttest("结婚的和尚未结婚的")
+    cuttest("结合成分子时")
+    cuttest("旅游和服务是最好的")
+    cuttest("这件事情的确是我的错")
+    cuttest("供大家参考指正")
+    cuttest("哈尔滨政府公布塌桥原因")
+    cuttest("我在机场入口处")
+    cuttest("邢永臣摄影报道")
+    cuttest("BP神经网络如何训练才能在分类时增加区分度？")
+    cuttest("南京市长江大桥")
+    cuttest("应一些使用者的建议，也为了便于利用NiuTrans用于SMT研究")
+    cuttest('长春市长春药店')
+    cuttest('邓颖超生前最喜欢的衣服')
+    cuttest('胡锦涛是热爱世界和平的政治局常委')
+    cuttest('程序员祝海林和朱会震是在孙健的左面和右面, 范凯在最右面.再往左是李松洪')
+    cuttest('一次性交多少钱')
+    cuttest('两块五一套，三块八一斤，四块七一本，五块六一条')
+    cuttest('小和尚留了一个像大和尚一样的和尚头')
+    cuttest('我是中华人民共和国公民;我爸爸是共和党党员; 地铁和平门站')
+    cuttest('张晓梅去人民医院做了个B超然后去买了件T恤')
+    cuttest('AT&T是一件不错的公司，给你发offer了吗？')
+    cuttest('C++和c#是什么关系？11+122=133，是吗？PI=3.14159')
+    cuttest('你认识那个和主席握手的的哥吗？他开一辆黑色的士。')
+    cuttest('枪杆子中出政权')
+    cuttest('张三风同学走上了不归路')
+    cuttest('阿Q腰间挂着BB机手里拿着大哥大，说：我一般吃饭不AA制的。')
+    cuttest('在1号店能买到小S和大S八卦的书，还有3D电视。')