Make `padding_mode` as Enum and strings
Open
Nobody has claimed this yet.
enhancement
module: transforms
prototype
- Dominant language
- Python
- Stars
- 17.9k
- Forks
- 7.3k
- Avg merge
- 1d 15h
- Merged PRs (30d)
- 13
Description
Currently padding_mode is a string: "constant", "edge", "reflect", "symmetric" and in the codebase we already have enums like InterpolationMode and others in prototype space.
Let's create an enum for padding mode and convert the type into it while keeping BC by supporting string values.
cc @vfdev-5 @datumbox @bjuncek @pmeier
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the existing prototype-space InterpolationMode and other enums, then find every use of padding_mode in the transforms code. Define the padding-mode enum with the listed values while preserving support for the existing string values, and run the affected tests to verify both forms remain supported.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- computer-vision
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100