-
Notifications
You must be signed in to change notification settings - Fork 304
The official Tensorflow implementation is now available. #134
Comments
But in tensorflow |
Yes unfortunately but it can be used to correct the numerous problems with all the public PyTorch implementations. Like for example, the original tensorflow code has three convolution layers per feature level in the bifpn, two 1x1 layers that are used to bring them into the same feature space on the upwards and downwards passes respectively and one Depthwise separable one that is applied after a BiFPN step (the one made explicit in the BiFPN math in the paper) |
The tensorflow code also confirmed my assumption that every convolution in the fpn+box network is either 1x1 or depthwise separable |
Nice, can you make a PR to this repo to fix the issue ? |
@rmcavoy I would suggest you fork this repo and create a modified version. Author of this repo seems to have disappeared. Link that repo here |
Unfortunately my ability to directly contribute code to open source repos is limited by my work (pretty sure I would need to do various kinds of paperwork for approval of such contributions) |
FYI, I decided to get a version of EfficientDet running now that an official reference is available. I saw this issue searching through the pre-existing PyTorch impls out there for a starting point. None of them fit with my approach so I started from scratch earlier this week. Model is 80% done, then I'll hack together a weight port and a basic validation script. Might have something to release later next week... EDIT: This is where it will live when it's done (nothing there yet but the plan) - https://github.com/rwightman/efficientdet-pytorch |
That would be wonderful ! The google's repo is awful to use in a research-oriented env, even only extracting the model itself is tedious and the other Pytorch implementation out there are not convincing. Thank you very much |
@rwightman Could you please make the segmentation model as well, it is really simple, the paper just says
It would be super helpful as even google has not released it |
https://github.com/google/automl/blob/master/efficientdet/efficientdet_arch.py
The text was updated successfully, but these errors were encountered: