-
-
Notifications
You must be signed in to change notification settings - Fork 216
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
Viral Shorts Video Analysis #627
Conversation
Our team will soon review your PR. Thanks @HarshRaj29004 :) |
Now i have done all sort of changes |
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.
- remove the
bfg.jar
file. - Sub folder names should be like,
Models
,Images
,Dataset
. - It's not
required.txt
, it should berequirements.txt
.
Please update the sub folder names, it should be Dataset, Images like this. |
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.
Hi @HarshRaj29004 visual plots are not showing after you made your changes.
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.
Approved @HarshRaj29004
Hi @HarshRaj29004 please share your email address for further communications. |
Pull Request for ML-Crate 💡
Issue Title: Viral Shorts Video Analysis
Closes: #509
Describe the add-ons or changes you've made 📃
Give a clear description of what have you added or modifications made
Type of change ☑️
What sort of change have you made:
How Has This Been Tested? ⚙️
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑️