Skip to content

feat: added product detection for both video and image #64

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

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

Conversation

ARYPROGRAMMER
Copy link
Contributor

Description

This PR adds the following:

Implement image analysis
Add feature detection
Create classification system
Add confidence scoring
Implement result validation

Related Issue

Fixes #26

Motivation and Context

Core necessity of the application

How Has This Been Tested?

Tested locally on wsl virtual env

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist:

  • My code follows the code style of this project
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • My changes generate no new warnings
  • I have checked my code and corrected any misspellings

Signed-off-by: Arya Pratap Singh <notaryasingh@gmail.com>
@Varsha-1605
Copy link
Owner

@ARYPROGRAMMER show it's working and make sure it works after deployment as well not just for local systems.

@Varsha-1605
Copy link
Owner

@ARYPROGRAMMER ???

@ARYPROGRAMMER
Copy link
Contributor Author

@ARYPROGRAMMER ???

Hey @Varsha-1605 I am not well currently, this I am unable to start my laptop for now. Provide me some time I'll give you the working. If it's getting late maybe you can test this commit locally and let me know further thanks

@Varsha-1605
Copy link
Owner

@ARYPROGRAMMER Alright, Take care!

@Varsha-1605
Copy link
Owner

@ARYPROGRAMMER I hope you are well!
kindly share the video of your work !

@ARYPROGRAMMER
Copy link
Contributor Author

@ARYPROGRAMMER I hope you are well! kindly share the video of your work !

yup back , allow me some time i'll share a video

@Varsha-1605
Copy link
Owner

@ARYPROGRAMMER this is the last time m asking to share the video else I'll close the pr

@ARYPROGRAMMER
Copy link
Contributor Author

@ARYPROGRAMMER this is the last time m asking to share the video else I'll close the pr

Apologies @Varsha-1605 , the current implementation was not working and i am unable to work on it currently due to medical reasons. Hope you understand

@Varsha-1605
Copy link
Owner

@ARYPROGRAMMER ok!

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

Successfully merging this pull request may close these issues.

[Custom Issue]: Implement Product Detection
2 participants