Skip to content

Dropout (Grid) and Dropout (Cutout) on HIP and HOST#679

Merged
LakshmiKumar23 merged 139 commits into
ROCm:developfrom
r-abishek:ar/dropout_grid_cutout
Mar 23, 2026
Merged

Dropout (Grid) and Dropout (Cutout) on HIP and HOST#679
LakshmiKumar23 merged 139 commits into
ROCm:developfrom
r-abishek:ar/dropout_grid_cutout

Conversation

@r-abishek
Copy link
Copy Markdown
Member

  • Adds Grid and Cutout Dropout augmentations on HIP and HOST
  • Adds support for U8/F32/F16/I8 bit depths and NCHW/NHWC variants with toggle support
  • Adds relevant unit / qa / performance tests

snehaa8 and others added 30 commits August 22, 2025 07:39
…ied QA for logic to implement random seed for non QA
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 25, 2026

Codecov Report

❌ Patch coverage is 97.53954% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...dules/tensor/rppt_tensor_effects_augmentations.cpp 95.56% 12 Missing ⚠️
src/modules/tensor/hip/kernel/grid_dropout.cpp 97.75% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #679      +/-   ##
===========================================
- Coverage    92.49%   91.96%   -0.52%     
===========================================
  Files          202      204       +2     
  Lines        91730    92299     +569     
===========================================
+ Hits         84838    84882      +44     
- Misses        6892     7417     +525     
Files with missing lines Coverage Δ
src/modules/tensor/cpu/kernel/grid_dropout.cpp 100.00% <100.00%> (ø)
src/modules/tensor/hip/kernel/grid_dropout.cpp 97.75% <97.75%> (ø)
...dules/tensor/rppt_tensor_effects_augmentations.cpp 89.50% <95.56%> (-3.40%) ⬇️

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread src/modules/tensor/cpu/kernel/grid_dropout.cpp Outdated
Comment thread src/modules/tensor/hip/kernel/grid_dropout.cpp Outdated
Copy link
Copy Markdown
Contributor

@rrawther rrawther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please address all the review comments

Comment thread src/modules/tensor/hip/kernel/grid_dropout.cpp
Comment thread src/modules/tensor/cpu/kernel/grid_dropout.cpp
Comment thread src/modules/tensor/cpu/kernel/grid_dropout.cpp Outdated
Comment thread src/modules/tensor/hip/kernel/grid_dropout.cpp Outdated
Copy link
Copy Markdown
Contributor

@rrawther rrawther left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please address review comments

@LakshmiKumar23 LakshmiKumar23 merged commit 74e2209 into ROCm:develop Mar 23, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:precheckin enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants