David Slater

Should be integrated into ART 1.11

How about we simply enforce `batch_size > 1` for this? (And give a small note about the ART issue.)

There doesn't seem to be a good way of fixing this on our end, as examples can get filtered out, leaving it possible for this error to arise nondeterministically. The...

I think the only reason this was done was to log a warning if the armory launcher has a different version than armory running the scenario: https://github.com/twosixlabs/armory/blob/master/armory/scenarios/main.py#L211-L216

There isn't a clean distinction or API at the moment. Probably will become less of an issue once we do a better job of separating the two.

Rename to `CLIENT_ARMORY_VERSION`? Used to log a warning if the version used in the container differs from the version used in the client (since we don't have an API versioning...

Thanks, @lcadalzo, I hadn't tested with model predictions.

Related to #1619

I think it could really simplify a lot of our command line, config parsing, python object loading, and config verification.

If we are going to move in this direction, let's also remove `tf2` and `pytorch-deepspeech`, and just move directly to one container.