Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Spot add ldaonlineoptimizer support #3

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Spot add ldaonlineoptimizer support #3

wants to merge 6 commits into from

Commits on May 22, 2017

  1. Adding support for OnlineLDAOptimizer. Still need to work on TODOs.

    Ricardo Barona committed May 22, 2017
    Configuration menu
    Copy the full SHA
    c99070f View commit details
    Browse the repository at this point in the history

  2. Unit test fails for OnlineOptimizer, need to fix

    Ricardo Barona committed May 22, 2017
    Configuration menu
    Copy the full SHA
    dcc0a99 View commit details
    Browse the repository at this point in the history

  3. Adding OnlineLDAOptimizer support

    Ricardo Barona committed May 22, 2017
    Configuration menu
    Copy the full SHA
    01351fc View commit details
    Browse the repository at this point in the history

Commits on May 26, 2017

  1. Added unit testing test configuration for online optimizer in DNS and… 

    … Proxy implementations.

Refactored code in SpotLDAWrapper to implement one or the other LDA optimizers.
    Ricardo Barona committed May 26, 2017
    Configuration menu
    Copy the full SHA
    895d5a4 View commit details
    Browse the repository at this point in the history

Commits on May 30, 2017

  1. Updated ml_ops.sh and ml_test.sh to include LDA parameters: optimizer… 

    …, alpha and beta.

Updated spot.conf to include the same parameters.
    Ricardo Barona committed May 30, 2017
    Configuration menu
    Copy the full SHA
    bb52a9a View commit details
    Browse the repository at this point in the history

  2. Made changes to SpotLDAWrapper and ml_ops.sh based on feedback from @… 

    …NathanSegerlind

Fixed inline comments format, added one space after //.
    Ricardo Barona committed May 30, 2017
    Configuration menu
    Copy the full SHA
    94d812c View commit details
    Browse the repository at this point in the history