Denisa Roberts
Denisa Roberts
@Rocketknight1 @gante This PR is now ready for review.
@Rocketknight1 @amyeroberts I addressed your comments and also submitted two PRs for the l1 and l3 weights (and tagged Rocketknight1). Let me know what's next!
@amyeroberts Thanks for pointing out the circle ci fix. It appears that one doc test which (rightly) can't find tf weights is failing for now. I added back the `from_pt`...
@amyeroberts @Rocketknight1 All local tests pass with the new tf weights. The CI gets this documentation tests failing; the pt version also predicts 281 which maps to label_281 in config.
@Rocketknight1 @amyeroberts Alright. :)
Hi @Rocketknight1 , I've addressed the last comments from @amyeroberts and had all tests pass. I am ready for merge whenever you are. I've just rebased to upstream and there...
@Rocketknight1 All green again. :)
@sgugger @amyeroberts @Rocketknight1 I was wondering - when do you plan a transformers release that includes this code?
Are there plans for tensorflow version? I am interested in having the tf model available for use in a downstream task in my work/research
@amyeroberts I am still working on porting the Efficientformer; I am interested in having both in tf to train in some downstream tasks / research... I would like to do...