WIP: Efficient Detection Net

[danielkurniadi]

Before submitting

• Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
• Did you read the contributor guideline, Pull Request section?
• Did you make sure to update the docs?
• Did you write any new necessary tests?

What does this PR do?

Implement #1: #1

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[pep8speaks]

Hello @iqDF! Thanks for updating this PR.

• In the file pl_bolts/models/vision/bifpn.py:

Line 42:121: E501 line too long (128 > 120 characters)
Line 43:1: W293 blank line contains whitespace
Line 46:1: W293 blank line contains whitespace
Line 74:21: E128 continuation line under-indented for visual indent
Line 86:1: W293 blank line contains whitespace
Line 96:1: W293 blank line contains whitespace
Line 101:27: E261 at least two spaces before inline comment
Line 102:1: W293 blank line contains whitespace
Line 108:55: E226 missing whitespace around arithmetic operator
Line 109:44: E261 at least two spaces before inline comment
Line 111:1: W293 blank line contains whitespace
Line 115:1: W293 blank line contains whitespace
Line 122:80: W291 trailing whitespace
Line 123:62: W291 trailing whitespace

Comment last updated at 2020-06-28 03:40:51 UTC

[danielkurniadi]

@pep8speaks Will fix Pep8 linting in the next commit.

[williamFalcon]

@iqDF fixed the conflicts!
this is super cool. let's get this merged

[williamFalcon]

@iqDF don't worry about builds or any of that right now. let's just get the model working (confirmed). we can work out the build issues later

[danielkurniadi]

@williamFalcon ok cool. will continue the work.

[mergify]

This pull request is now in conflict... :(

[williamFalcon]

@iqDF ok, we now merged a lot of the building blocks and structures.
Take a look at some of the self-supervised examples for the standard conventions :)

looking forward to adding!

[williamFalcon]

@iqDF we finally got the v1 of bolts ready with tests and docs. Let's continue this implementation?
I noticed there is a timm dependency? mind implementing these things directly since we want to keep bolts lightweight?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.