Multiple crops example

[mzient]

Category:

Other (e.g. Documentation, Tests, Configuration)

Description:

Implements a pipeline example requested by a user in #1579

Additional information:

Affected modules and functionalities:

N/A

Key points relevant for the review:

Tests:

• [ ] Existing tests apply
• [ ] New tests added
  • [ ] Python tests
  • [ ] GTests
  • [ ] Benchmark
  • [ ] Other
• [X] N/A

Checklist

Documentation

• [ ] Existing documentation applies
• [X] Documentation updated
  • [ ] Docstring
  • [ ] Doxygen
  • [ ] RST
  • [X] Jupyter
  • [ ] Other
• [ ] N/A

DALI team only

Requirements

• [ ] Implements new requirements
• [ ] Affects existing requirements
• [X] N/A

REQ IDs: N/A

JIRA TASK: N/A

[review-notebook-app[bot]]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[dali-automaton]

CI MESSAGE: [18958041]: BUILD STARTED

[dali-automaton]

CI MESSAGE: [18958465]: BUILD STARTED

[dali-automaton]

CI MESSAGE: [18958465]: BUILD PASSED

[stiepan]

For the future, I belive we could make the resize and crop ops support similar trick, likely in a bit more straightforward way.

[review-notebook-app[bot]]

View / edit / reply to this conversation on ReviewNB

szkarpinski commented on 2024-10-04T11:44:12Z ----------------------------------------------------------------

Line #42.        )  # convert to float and mark outermost dimension as"frame"

Nitpicks: Why +0.0 and not cast? Also, there's a missing space before "frame"

[review-notebook-app[bot]]

View / edit / reply to this conversation on ReviewNB

szkarpinski commented on 2024-10-04T11:44:13Z ----------------------------------------------------------------

How about adding a sentence explaining what the images above mean? The first single image was confusing to me - I expected sequences of equal length. It took me some time to notice num_crops = random (...) in the code.

mzient commented on 2024-11-04T11:46:51Z ----------------------------------------------------------------

Yeah, I'd recommend playing with the seed a bit to get an example with three different sequence lengths, preferably without a single-image sequence.

[mzient]

Yeah, I'd recommend playing with the seed a bit to get an example with three different sequence lengths, preferably without a single-image sequence.

---

View entire conversation on ReviewNB