Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ops/image face blur mapper #249

Merged
merged 6 commits into from
Mar 14, 2024
Merged

ops/image face blur mapper #249

merged 6 commits into from
Mar 14, 2024

Conversation

drcege
Copy link
Collaborator

@drcege drcege commented Mar 13, 2024

Image OPs

  • add image_face_blur_mapper
  • rename face_area_filter to image_face_ratio_filter

Video OPs

  • will add video_face_blur_mapper in another PR
  • is it necessary to add video_face_ratio_filter? @yxdyc @HYLcool

@drcege drcege added enhancement New feature or request dj:multimodal issues/PRs about multimodal data processing dj:op issues/PRs about some specific OPs labels Mar 13, 2024
@drcege drcege added this to the DJ-SORA milestone Mar 13, 2024
@drcege drcege self-assigned this Mar 13, 2024
@drcege drcege linked an issue Mar 13, 2024 that may be closed by this pull request
Video content compliance and privacy protection operators (image, text, audio) #233
Closed
HYLcool
HYLcool reviewed Mar 14, 2024
View reviewed changes
docs/Operators.md Outdated Show resolved Hide resolved
HYLcool
HYLcool reviewed Mar 14, 2024
View reviewed changes
Copy link
Collaborator

@HYLcool HYLcool left a comment

Choose a reason for hiding this comment

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

Besides, we need to add this new op to the sphinx docs like this PR does.

HYLcool
HYLcool approved these changes Mar 14, 2024
View reviewed changes
Copy link
Collaborator

@HYLcool HYLcool left a comment

Choose a reason for hiding this comment

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

LGTM

@drcege
Copy link
Collaborator Author

drcege commented Mar 14, 2024

Besides, we need to add this new op to the sphinx docs like this PR does.

added

@drcege drcege merged commit 59007f4 into main Mar 14, 2024
3 checks passed
@HYLcool HYLcool deleted the ops/image_face_blur_mapper branch December 23, 2024 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HYLcool HYLcool HYLcool approved these changes

@chenhesen chenhesen Awaiting requested review from chenhesen

@BeachWang BeachWang Awaiting requested review from BeachWang

@zhijianma zhijianma Awaiting requested review from zhijianma

@yxdyc yxdyc Awaiting requested review from yxdyc

Assignees

@drcege drcege

Labels
dj:multimodal issues/PRs about multimodal data processing dj:op issues/PRs about some specific OPs enhancement New feature or request
Projects
None yet
Milestone
DJ-SORA
Development

Successfully merging this pull request may close these issues.

Video content compliance and privacy protection operators (image, text, audio)
2 participants
@drcege @HYLcool