-
-
Notifications
You must be signed in to change notification settings - Fork 166
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.
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
Created and implemented Overlay face function #3241
base: main
Are you sure you want to change the base?
Conversation
I'm concerned about the memory usage of numpy, is there a way to know how much more memory this will consume? |
@DonnieBLT sir, I'm unsure of why pre-commit test is failing continuously even though I'm running pre-commit everytime before committing the changes. Please suggest what should I do to resolve this, thank you! ![]() |
I've added |
Can you please add an automated test to this so that it starts with a face, then upload it to an issue and then check the issue and make sure that it does not detect a face |
It’s precommit formatting |
sure sir, I'll do that. thank you! |
@DonnieBLT sir, can you please explain what was required to be done here? |
Create a test |
@DonnieBLT sir, please review the changes. thank you! Screen.Recording.2025-02-05.at.5.46.25.PM.mov |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove unused, dependencies, and separate the functionality into a new PR if it’s still there
@DonnieBLT ![]() after downloading dependencies: (opencv-python, numpy): ![]() |
Closes: #2527
#3243
I've also made changes to
delete_issue
function to reduce time complexity and better optimisationScreen.Recording.2025-01-19.at.6.10.46.PM.mov