Skip to content

Conversation

@SundarRajan98
Copy link
Contributor

@SundarRajan98 SundarRajan98 commented Jan 9, 2026

Motivation

Integrates new RPP augmentations in rocAL:

  • GaussianNoise
  • ShotNoise
  • Spatter
  • Log
  • ColorJitter
  • Water

Corresponding MIVisionX PR - #1601

Technical Details

  • Added six new augmentation kernels with both fixed and parameterized variants
  • Implemented C++ node classes for each augmentation with proper initialization and update methods
  • Exposed Python bindings for all new augmentations

Test Plan

Test Result

Submission Checklist

…Spatter, Log, ColorJitter, Water

- Add node headers and implementations for GaussianNoise, ShotNoise, Spatter, Log, ColorJitter, Water
- Add C API declarations and implementations in rocal_api_augmentation.h/cpp
- Add Python bindings in rocal_pybind.cpp and fn.py
- Update VX_EXT_RPP_CHECK_VERSION to (3, 1, 5) in API file
- Update THROW messages to indicate vx_rpp >= 3.1.5
- Add version checks in node files: node_gaussian_noise.cpp,
  node_shot_noise.cpp, node_spatter.cpp, node_water.cpp,
  node_color_jitter.cpp, node_log.cpp
C++ tests (unit_tests.cpp):
- Case 64: rocalGaussianNoise (random)
- Case 65: rocalGaussianNoiseFixed
- Case 66: rocalShotNoise (random)
- Case 67: rocalShotNoiseFixed
- Case 68: rocalSpatter
- Case 69: rocalLog
- Case 70: rocalColorJitter (random)
- Case 71: rocalColorJitterFixed
- Case 72: rocalWater (random)
- Case 73: rocalWaterFixed

Python tests (unit_test.py):
- gaussian_noise, shot_noise, spatter, water, color_jitter

Shell scripts updated:
- testAllScripts.sh: Added C++ test entries
- unit_tests.sh: Added Python test entries
SundarRajan98 and others added 4 commits January 10, 2026 13:08
Move version check to the very beginning of each function,
before any variable assignments or context validation, for
all Set 1 augmentation APIs.
@kiritigowda kiritigowda self-assigned this Jan 20, 2026
@kiritigowda kiritigowda added enhancement New feature or request ci:precheckin labels Jan 20, 2026
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.

3 participants