search

messense / jieba-rs

The Jieba Chinese Word Segmentation Implemented in Rust
MIT License
738 stars 46 forks source link

Remove char_indices from hmm's code #45

Closed MnO2 closed 5 years ago

MnO2 commented 5 years ago 
jieba cut no hmm        time:   [6.4076 us 6.4389 us 6.4909 us]
                        change: [-3.2786% -1.8715% -0.3329%] (p = 0.01 < 0.05)
                        Change within noise threshold.
Found 12 outliers among 100 measurements (12.00%)
  3 (3.00%) high mild
  9 (9.00%) high severe

jieba cut with hmm      time:   [8.5764 us 8.5923 us 8.6135 us]
                        change: [-12.435% -11.527% -10.554%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 11 outliers among 100 measurements (11.00%)
  1 (1.00%) high mild
  10 (10.00%) high severe

jieba cut_all           time:   [5.0658 us 5.0958 us 5.1550 us]
                        change: [-17.869% -16.887% -15.722%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 13 outliers among 100 measurements (13.00%)
➜  jieba-rs git:(remove-hmm-charindices) ./target/release/weicheng
3959ms
codecov[bot] commented 5 years ago

Codecov Report

Merging #45 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #45   +/-   ##
=======================================
  Coverage   96.73%   96.73%           
=======================================
  Files           3        3           
  Lines         184      184           
=======================================
  Hits          178      178           
  Misses          6        6

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bc41b52...7b87311. Read the comment docs.