Brendan Roof

We'd certainly be open to taking a PR for this assuming the issue you highlight can be resolved in a clean fashion. The other issue that we're unsure about is...

Thanks for the offer, @bryant1410! I've mostly been using the `NoOpTrainer` and then `evaluat`ing separately, but we think what you've outlined makes sense. We'd be happy to review your PR....

Hi @mchari, `logging.DEBUG` is a lower level than `logging.INFO` so it includes all of the info messages. One thing we could do here would be to have an environment variable...

Hi @Prhmma, thanks for offering! The logic appears to exist in https://github.com/allenai/allennlp/blob/3dda5ac9fcf390d8b83eb855249360711707c11c/allennlp/run.py#L6. You'll just need to add a new environment variable for the log level and add a few conditionals....

This would be cool! @matt-gardner mentioned wanting this just the other day, actually. Could you ping this issue when the underlying bug is resolved? Contributions are most certainly welcome...

@kernelmachine , could you take a look at this? (As it looks like you're the author of the `BagOfWordCountsTokenEmbedder`.) @camelop's point seems like a good one, but I'm not sure...