hmc-whisk / jsLDA

A React based version of jsLDA with brand new features added on
Other
0 stars 0 forks source link

Fixed the problem #86

Closed theobayard closed 4 years ago

theobayard commented 4 years ago

This closes #85. I left the d3 in because, while not perfectly inline with React, it is by far the easiest way to keep the iterations updated with the LDAModel. Otherwise we'd have to build another callback into the LDAModel and then that call back might end up re rendering the whole page and that would be a whole other thing to deal with. The current method works, and doesn't seem to cause any glitches.

mimno commented 4 years ago

Not surprised, getting a non-blocking iteration display update took a long time to figure out!