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

Integrated features for depression detection #264

Conversation

Nikhil210206
Copy link

@Nikhil210206 Nikhil210206 commented Jan 14, 2025

Closes: #55

Title: Integrated Depression Detection Features (Text, Face, and Audio)
Your Name: Nikhil Balamurugan
Describe the add-ons or changes you've made 📃:
In this PR, I have integrated multiple depression detection features into the project:
Text-based Depression Detection: Added NLP model for analyzing text input and detecting depression.
Face-based Depression Detection: Integrated FER (Facial Expression Recognition) library to detect emotions from a user’s face using webcam input.
Checklist: ☑️

My code follows the Contributing Guidelines & Code of Conduct of this project.
This PR does not contain plagiarized content.
I have performed a self-review of my own code.
I have commented my code, particularly wherever it was hard to understand.
My changes generate no new warnings.

Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

Copy link
Owner

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

Nice Work! Remove all .DS_Store files and integrate it with Jarvis rather than building a flask app.

  • Also submit a working video after integration with Jarvis.
  • Link your issue with pr.

@Avdhesh-Varshney Avdhesh-Varshney added bug 🐛 Something isn't working Status: Requested Changes Warning: Follow Guidelines https://github.com/Avdhesh-Varshney/Jarvis/issues/113 labels Jan 14, 2025
@jindalpriyanshu101
Copy link
Collaborator

@Nikhil210206 Kindly link your issue as well. Follow the PR Guidelines.

@Nikhil210206
Copy link
Author

I am not able to link it with Jarvis

@Nikhil210206
Copy link
Author

my issue is linked with this

@Nikhil210206
Copy link
Author

??

@Avdhesh-Varshney
Copy link
Owner

I am not able to link it with Jarvis

You should discuss about the issue on discord to get quick response.
If you having any error, you can also raise a QA discussion channel in Jarvis.

@Nikhil210206
Copy link
Author

send your username , the one in your profile says it is incorrect

@Avdhesh-Varshney
Copy link
Owner

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

Join it.

@Nikhil210206
Copy link
Author

joined

Copy link
Collaborator

@jindalpriyanshu101 jindalpriyanshu101 left a comment

Choose a reason for hiding this comment

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

Hey @Nikhil210206 as stated earlier, kindly remove all .DS_Store files from this PR.

Also, instead of building a flask app, use streamlit to integrate this depression detection with Jarvis.

You can go through codebase to learn how other models are implemented using streamlit into Jarvis to get a rough idea. Also make sure to go through Contribution Guidelines, it'll help you alot with naming conventions etc.

@Nikhil210206
Copy link
Author

I am not able to delete or remove those .DS files. I even checked in my vs code, those files are not there, it only appears in the pr I dont know why

@jindalpriyanshu101
Copy link
Collaborator

I am not able to delete or remove those .DS files. I even checked in my vs code, those files are not there, it only appears in the pr I dont know why

It should be there as hidden files. Enable the options to view hidden files, you'll find it.

Otherwise, revert your commit, exclude all these .DS_Store files from staging and recommit the changes.

@Nikhil210206
Copy link
Author

also I have used streamlit instead of flask

@Nikhil210206
Copy link
Author

just now I updated the code, please check

@Nikhil210206 Nikhil210206 force-pushed the feature/depression-detection-integration branch from a1c6de8 to 23484c6 Compare January 17, 2025 18:40
@Nikhil210206
Copy link
Author

now I have deleted the .DS files

Copy link
Collaborator

@jindalpriyanshu101 jindalpriyanshu101 left a comment

Choose a reason for hiding this comment

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

Hey @Nikhil210206, While going through your updated changes, i have left my comments on each files. Kindly get the requested stuffs done.

Apart from this, You should keep filename and your main function-name` same, in camelCase. (I suggest you to read and follow contribution guidelines)

data/Refute C.docx Outdated Show resolved Hide resolved
DepressionDetection/requirements.txt Outdated Show resolved Hide resolved
DepressionDetection/templates/index.html Outdated Show resolved Hide resolved
DepressionDetection/templates/result.html Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
DepressionDetection/app.py Outdated Show resolved Hide resolved
@Nikhil210206
Copy link
Author

Screen.Recording.2025-01-19.at.11.33.44.AM.mov

@Nikhil210206
Copy link
Author

I have made the changes and edited the file, now it works properly

@Avdhesh-Varshney Avdhesh-Varshney added wontfix ❌ This will not be worked on and removed Status: Requested Changes labels Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working Warning: Follow Guidelines https://github.com/Avdhesh-Varshney/Jarvis/issues/113 wontfix ❌ This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

📃: Depression detection NLP model
3 participants