BinPro / CONCOCT

Clustering cONtigs with COverage and ComposiTion
Other
119 stars 48 forks source link

added 5th argument for vbgmm, using .values #284

Open Sebastien-Raguideau opened 4 years ago

Sebastien-Raguideau commented 4 years ago

Hi Johannes

Somehow followup of issue #272, I changed all the deprecated call to as_matrix with the .values. I think the current deveop branch has .to_numpy() method, I tried that once and it did not work.

I also chanced upon another issue, since last year vbgmm ask for a 5th argument, the number of iterations. I copied/paste the same flag/default as concoct for number of iterations and added it to vbgmm.fit .

Best,

Seb