DALI
DALI copied to clipboard
Add enumeration of supported border modes
Signed-off-by: Kamil Tokarski [email protected]
Category:
Other (e.g. Documentation, Tests, Configuration)
Description:
Adds enum type with border modes supported by the filter (convolution) operator. The enum must be exposed in Python so that it can be used as a value passed for operator's border_mode
argument.
Additional information:
Affected modules and functionalities:
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
- [ ] Documentation updated
- [ ] Docstring
- [ ] Doxygen
- [ ] RST
- [ ] Jupyter
- [ ] Other
- [x] N/A
DALI team only
Requirements
- [ ] Implements new requirements
- [ ] Affects existing requirements
- [x] N/A
REQ IDs: N/A
JIRA TASK: DALI-2949
!build
CI MESSAGE: [5969400]: BUILD STARTED
CI MESSAGE: [5969400]: BUILD FAILED
!build
CI MESSAGE: [5969480]: BUILD STARTED
CI MESSAGE: [5969480]: BUILD PASSED
After some discussion I decided to close this one in favour of:
- using plain strings in python API (so no enum exposure through pybind),
- in the meantime @szalpal added similar enum, sticking to it. https://github.com/NVIDIA/DALI/pull/4284
https://github.com/NVIDIA/DALI/pull/4298