Add mask processing to COCO Reader with Augmentations example

[jantonguirao]

Signed-off-by: Joaquin Anton janton@nvidia.com

Why we need this PR?

Pick one, remove the rest

• It enhances the "COCO Reader with Augmentations" example, demonstrating mask polygon processing

What happened in this PR?

Fill relevant points, put NA otherwise. Replace anything inside []

• What solution was applied:
  Added mask polygon processing to the example
• Affected modules and functionalities:
  COCO Reader with Augmentations example
• Key points relevant for the review:
  All
• Validation and testing:
  N/A
• Documentation (including examples):
  N/A

JIRA TASK: [Use DALI-XXXX or NA]

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

banasraf commented on 2020-11-03T09:45:27Z
----------------------------------------------------------------

We will start...

Later you write "We can now run the pipeline". Not sure if it's inconsistency in the tense. Anyway, I personally prefer present tense in the examples but I guess both are ok if it's consistent.

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

banasraf commented on 2020-11-03T09:45:27Z
----------------------------------------------------------------

have to been -> have to be

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

banasraf commented on 2020-11-03T09:45:28Z
----------------------------------------------------------------

If necessary, we could clamp the coordinates

If necessary, clamp the coordinates ?

Comments describing the operators also could be more consistent. There's "Converting bounding boxes..." and "Adjust masks coordinates...". I would go for imperative but gerund is also ok, I think.

relative to absolute coordinates by mapping the top-l comment lacks indentation

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

JanuszL commented on 2020-11-03T18:02:39Z
----------------------------------------------------------------

Do we need "# Using DALI's functional style API" comment?

[review-notebook-app]

View / edit / reply to this conversation on ReviewNB

JanuszL commented on 2020-11-03T18:02:40Z
----------------------------------------------------------------

Wrapping the pipeline definition in separate functions that we can reuse it later

->

# Wrapping the pipeline definition in separate functions that we can reuse later

[jantonguirao]

!build

[dali-automaton]

CI MESSAGE: [1765255]: BUILD STARTED

[dali-automaton]

CI MESSAGE: [1765255]: BUILD PASSED