search

antoniobruto / PSIMiner

Miner for Learning Prefix Sequences as Explanations for Events in time-series
GNU General Public License v3.0
2 stars 1 forks source link

Eliminate Redundant Computations #13

Open antoniobruto opened 3 years ago

antoniobruto commented 3 years ago

Eliminate redundant computations:

  1. Mean
  2. Error
  3. EndMatches
  4. Buckets
  5. Forward Influence

Store all this information in the node.

antoniobruto commented 3 years ago

Issue branch "PSI-E-1" created.

antoniobruto commented 3 years ago

Merged to master

Replaced inline code in amsMine : (1) Check for assertion node: is_assertion_node() , (2) Check for terminal node: function stop_learn(), (3) Preparing the node: function prepareTreeNode(), (4) Prepare constraint list: function createTruthListFalse(), createTruthListTrue()

antoniobruto commented 3 years ago

Breaking up issue into parts - label [codeRefactor]

  1. Refactor End Match Computations
  2. Refactor Entropy Computations
  3. Refactor Mean Computations
  4. Code Cleanup - Delete unused and commented out code
  5. Rename functions to be consistent with PSIMiner theory
  6. Descriptive Comment Labels for functions