search

MojoJolo / textteaser

TextTeaser is an automatic summarization algorithm.
MIT License
1.97k stars 250 forks source link

computation in Summarizer.dbs method is wrong #24

Closed chengdujin closed 10 years ago

chengdujin commented 10 years ago

Original: (r._1._1 * r._2._2) / Math.pow(r._1._2 - r._2._2, 2)

Which should be: (r._1._1 * r._2._1) / Math.pow(r._1._2 - r._2._2, 2)

tkroman commented 10 years ago

I have already noticed it and fixed it in my PR, but @MojoJolo doesn't seem to be able to accept it right now.

MojoJolo commented 10 years ago

Didn't manage to check it right now. Sorry about it. @cdshines. I just saw your pull request. Weird because Github never alerts me.

a7i7 commented 8 years ago

Can anyone tell me what is dbs and sbs? I want to know the logic behind it