ajbc / trellis

Other
9 stars 3 forks source link

Optimize backend precomputations #108

Closed AFriendlyRobot closed 6 years ago

AFriendlyRobot commented 6 years ago

NOTE: At the time of creation, this branch calculates/updates some things more efficiently, but seems to hog memory.

AFriendlyRobot commented 6 years ago

The "memory hogging" was actually an infinite deadlocking attempt to initialize two values based on each other. That has been fixed.

AFriendlyRobot commented 6 years ago

HOWEVER: In this branch at least, there seems to be some bug related to clustering or updating theta/beta values improperly. I will not merge until this has been taken care of.

AFriendlyRobot commented 6 years ago

I believe this branch fixes the phantom topic issue (by correctly assigning leaves)