Ordering of intervals in RRA seems wrong

GrammarViz2 / grammarviz2_src

GrammarViz 2.0 public release:

http://grammarviz2.github.io/grammarviz2_site

GNU General Public License v2.0

120 stars 39 forks source link

Ordering of intervals in RRA seems wrong #40

Closed GabrieleCocco closed 8 years ago

GabrieleCocco commented 8 years ago

According to a paper the intervals in RRA should be ordered by increasing Usage Frequency, while in findBestDiscordForInterval they seem to be sorted based on coverage.

GabrieleCocco commented 8 years ago

Ok, possible explanation is the coverage set a bit counter-intuitively here

seninp commented 8 years ago

actually, there are more than these two ways to apply heuristics ordering, in GUI I was experimenting with these five in addition to the rule frequency:

heuristics for intervals ordering

you may try these for your own application domain as the results (the performance) may vary significantly.

kudos for looking into the code!