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.

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

Train and deploy rcnn and cnn model #116

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

devxMani
Copy link
Contributor

@devxMani devxMani commented Nov 9, 2024

#115

I've updated the MTCNN face detection module to better integrate with your SPROCTOR project.
Added proper class structure:-
1- FaceDetector class for MTCNN operations
2- MTCNNDetectionProcess class that inherits from your project's Process class

Improved error handling and logging:
Configured proper logging
Added try-except blocks for robust error handling

Can you Gssoc ext level 3 to this PR...

Copy link
Contributor

github-actions bot commented Nov 9, 2024

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@MastanSayyad
Copy link

@devxMani Not assigned created 6 hours back, so no points

image

@tanishaness
Copy link
Owner

@MastanSayyad please check

@devxMani
Copy link
Contributor Author

devxMani commented Nov 11, 2024

@tanishaness I gave him PR details for merging but now you can merge these

@devxMani
Copy link
Contributor Author

@tanishaness I have messaged you on Discord and LinkedIn but aap active hi nhi the, abb to waise bhi in PRs ke points nhi milege but aap merge karlo..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants