search

angeloskath / supervised-lda

A flexible variational inference LDA library.
MIT License
22 stars 5 forks source link

Default template argument for Scalar #15

Closed paschalidoud closed 7 years ago

angeloskath commented 7 years ago

I added a default value of double for LDA, LDABuilder and all the Parameters. @paschalidoud do you think this is enough or we need to add it to more places?

We can now create an LDA as simple as

LDA lda = LDABuilder().initialize_topics_random(100, 10);

Do you think it is bad that we hide the ability to use another Scalar type?