bt2901

Results 34 comments of bt2901

Yes, this is precisely what I needed for my use case. Huge thanks to you for implementing and publishing this. I have a question about `--category_surface` flag you added. I...

Thanks, that makes sense! I managed to hack something together, but I found myself confused about global state. You have class `Pipe` and object named `pipe`, an instance of it....

Did you mean `Union` instead of `Tuple`? Or am I confused about OR operator in `typing`?

I think you are moving the goalposts. We do not provide guarantees on `_fit`, but it does not forbid the user to use it. Making this method a bit more...

I agree with @Alvant, but I want to add another consideration. In many models, multiplying every modality weight by the same constant should leave the model unchanged (as a consequence,...

I noticed some problems with saving the regularizer as well; I need to track them down someday as well.

1) In our case, the practical benefits are insignificant (getting rid of installing superfluous libraries, but they are not heavy), but this seems to be a good practice. 2) I...

The way forward seems to distribute BigARTM with pip instead (building the library in a cross-platform way and bundling all external dependencies in `.whl` file). There's some progress on this...

The `messages_pb2.py` file is not included in this Github repository. It is generated as a part of the installation, but apparently this has failed in your case (for some reason)....

This is expected behaviour for hierarchical models. In that case, ARTM adds 'pseudodocuments' to the collection and tries to factorize this new collection. Each pseudodocument is related to some topic...