Jeff Picard
Jeff Picard
Thanks for running CI. I think the same error occurs on master. I think @helpmefindaname already fixed it in this [commit](https://github.com/flairNLP/flair/pull/3560/commits/783ebc7c5db43f2d0e3ca0d6473556da34a4e70e) but it hasn't merged yet. I've added that commit's...
Hey @HallerPatrick. Thanks again, I really appreciate your continued thought on this and bearing with me. Oops, sorry about that, I miscommunicated which script was the latest one to use....
I just rebased to pull in the type-checking updates to master and resolved conflicts. The guard wasn't working with non-serializable sentences so I added a commit fixing that (and removing...
Any thoughts on merging this @HallerPatrick @helpmefindaname?
For sure! Thank _you_ for building a framework that's so easy to love.
Many thanks for the review! I've squashed in a commit with your requested changes (Implement mini_batch_size and verbose; Rename loss). @elenamer would you be willing to take another look please?
(CI was failing with errors that looked unrelated to this branch so I clicked the "rebase" button in the UI)
Hi @alanakbik. I'm extremely sorry I took so long to reply, and many thanks for your thoughts. To your points, 1) Thanks for calling out the eval metrics confusion. I...
I would also love to see this fixed :)