Skip to content

Commit

Permalink
Remove TestLDAResults.test_expElogbeta()
Browse files Browse the repository at this point in the history
This test was using unexposed internal class methods in a way which was
incompatible with legitimate usage. It worked before due to how the
internals of the class were desiged, but no longer due to the fact that
the LDAResults._expElogbeta() method is const.
  • Loading branch information
ApproximateIdentity committed Oct 2, 2017
1 parent 96d4365 commit fd8d7c8
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions rosetta/tests/test_text.py
Original file line number Diff line number Diff line change
Expand Up @@ -323,16 +323,6 @@ def test_dirichlet_expectation(self):
[-0.13470677, -13.32429878]]).T
assert_allclose(result, benchmark, atol=1e-4)

def test_expElogbeta(self):
# Make sure equal to exponential of dirichlet_expectation when we
# pass in all ones
lda = self.choose_lda('lda')
lda._lambda_word_sums = pd.Series(
np.ones(lda.num_topics), index=lda.topics)
result = lda._expElogbeta
benchmark = np.exp(lda._dirichlet_expectation(lda.pr_token_topic))
assert_frame_equal(result, benchmark)

def test_predict_1(self):
# Use fact that w0 <--> topic_0, w1 <--> topic_1
lda = self.choose_lda('lda_2')
Expand Down

0 comments on commit fd8d7c8

Please sign in to comment.