Add typing annotations to models/detection

[oke-aditya]

🚀 The feature

This is an issue to track down all the files in detection module and work up the chain.

• _utils.py Add typing Annotations to detection/utils #4583
• anchor_utils.py Add typing to anchor utils #4599
• backbone_utils.py Add typing annotations to detection/backbone_utils #4603
• image_list.py Add typing annotations to detection/image_list #4602
• roi_heads.py Add typing annotations to detection/roi_heads #4612
• rpn.py Add typing annotations to detection/rpn #4619
• generalized_rcnn.py Add typing annotations to detection/generalized_rcnn  #4631
• transform.py Add typing annotations to detection/transform #4630

To type the models, all the above files would need to have annotations.

• faster_rcnn.py Add typing in detection module (faster rcnn). #4636
• mask_rcnn.py
• keypoint_rcnn.py
• retinanet.py
• ssd.py
• ssdlite.py

Motivation, pitch

@khushi-411 and I will sync and work on these files.

Solving This issue also helps #2025

As first step I'm closing #4220 and splitting it up into multiple PRs preferably one file each.

Hopefully we should complete this soon.

Alternatives

Note that there are cases where we have make JIT happy over mypy.

Most of these cases occur as JIT does not Support Tuple fully. In most cases we should prefer List.

Additional context

cc @datumbox @pmeier @NicolasHug

[datumbox]

Sounds goo.d I think SSD and SSDlite are mostly done. We are missing a couple of return types but parameters should be in place.

[khushi-411]

Let's start with other files too! @oke-aditya, have you started with any of them?

I'm moving ahead with backbone_utils.py. Please ping me if you have taken it up.

[oke-aditya]

Go ahead 😄 Just remember that after #4583 is merged you would need to remove that file from mypy.ini configuration.