search

ckmarkoh / deepSeg

A deep learning Chinese Word Segmentation toolkit
46 stars 18 forks source link

斷詞輸出格式回傳 list #5

Closed fukuball closed 8 years ago

fukuball commented 8 years ago

@ckmarkoh 這個 issue 先完成了，沒問題就 merge 吧～

drumrick commented 8 years ago

你應該可以merge吧？ 還是故意要另一個人code review再merge？

fukuball commented 8 years ago

@drumrick 故意讓另一人 code review，這樣比較好～

fukuball commented 8 years ago

覺得可能進展會太慢，不然我就先 merge，以後有需要導入 code review 制度再說好了～

drumrick commented 8 years ago

其實我有review, 回在這 https://github.com/ckmarkoh/deepSeg/issues/2#issuecomment-227321849 因為有點疑慮所以沒merge

fukuball commented 8 years ago

@drumrick 其實你有任何疑問就是直接在 pull request 上留言，這樣發布 request 的人就可以及時改，因為只要有 pull request 一直擱著，之後要 merge 起來就會麻煩（極有可能 conflict）

fukuball commented 8 years ago

@drumrick 對了，你知道 pull request 時，github 會將 change 列出來，然後可以在任何一行留 comment 嗎？所以其實要溝通哪裡有問題是非常容易的，如果有做任何 review 的動作，應該就是要在上面留下記錄，要善用工具