combust / combust/mleap

Reenable LDA parity spec

Open
#169 0 comments 0 reactions 0 assignees View on GitHub
bug help wanted
Dominant language
Scala
Stars
1.5k
Forks
315
PR merge metrics
No merged PRs in 30d

Description

LDA uses a random sampling to score data. This causes slightly different values to to be generated between each run, which makes testing outright equality of tensors a no-go for the parity spec.

We are temporarily disabling the LDA parity spec in favor of fixing tensor equality and will reenable the parity spec using lighter equality restrictions at a later time before the 0.7.0 release.

See the tensor equality issue here: https://github.com/combust/mleap/pull/156

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.