Janis Lesinskis

https://github.com/persephone-tools/persephone/blob/d73db4597a9ff92f6ea8824e6070b662a403655b/persephone/tests/experiments/test_na.py#L97-L122 Specifically on line 126 `get_simple_model` determines the batch size and passes it in as a parameter. Creating a simple test case like this could provide a nice regression test...

Added some tests for `utils.make_batch` in #189 and it appears the issue isn't from there. Will work on this more tomorrow.

I see in the constructor there's the following: ```python if not num_train: if not batch_size: batch_size = 64 num_train = len(corpus.get_train_fns()[0]) ``` This will find the `num_train` from how many...

This is one of those potentially difficult areas, let me know if you want any assistance.

Could you please provide us with the exact import error text?

Sounds like a versioning mismatch issue. Off the top of my head I'm not entirely sure if we can bump to `tensorflow-gpu>2` for the dependency in this library, but if...

One thing I'd request as someone who's contributed to this code is to either use a strong copy-left or the MPLv2 license. I'd explicitly like to avoid MIT license since...

I have expanded on my thoughts about the issues with the MIT license here: https://www.lesinskis.com/MIT-licence-community-hostile.html

It has been quite some time since I looked at the type stubs situation with tensorflow but I clearly remember the issues with the imports being a major challenge. It...

@Elynad I'd suggest that you use the logging functionality provided by flask to do this logging.