Skip to content

Doc issue fix 18836 FULL#19296

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
raaldrid:Doc_issue_fix_18836_FULL
Jan 11, 2021
Merged

Doc issue fix 18836 FULL#19296
opencv-pushbot merged 1 commit intoopencv:3.4from
raaldrid:Doc_issue_fix_18836_FULL

Conversation

@raaldrid
Copy link
Copy Markdown
Contributor

@raaldrid raaldrid commented Jan 8, 2021

Solution to documentation issue #18836
Clarify what FULL on enum means. H (hue) reduced on conversion to 0..180 (360/2) range by default for 8 bit images. FULL modifier expands H range to 0..255 for 8 bit images.

@asmorkalov asmorkalov added the category: documentation Documentation fix or update label Jan 11, 2021
@asmorkalov
Copy link
Copy Markdown
Contributor

The solution looks good to me. Could you squash git commits before merge to branch 3.4.

@asmorkalov asmorkalov self-requested a review January 11, 2021 12:16
@raaldrid raaldrid force-pushed the Doc_issue_fix_18836_FULL branch from 15706fa to f98e567 Compare January 11, 2021 18:45
@raaldrid
Copy link
Copy Markdown
Contributor Author

Thank you for your review. Commits squashed. Please let me know if I need to take further action.

Copy link
Copy Markdown
Contributor

@asmorkalov asmorkalov left a comment

Choose a reason for hiding this comment

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

👍

@opencv-pushbot opencv-pushbot merged commit 83e14f6 into opencv:3.4 Jan 11, 2021
@alalek alalek mentioned this pull request Jan 12, 2021
@alalek alalek mentioned this pull request Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: documentation Documentation fix or update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

undocumented *_FULL in cv::ColorConversionCodes

4 participants