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

enable MARLIN for emotion recognition on the ravdess and celebvhq dataset #24

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

Conversation

aromanusc
Copy link

@aromanusc aromanusc commented Feb 9, 2024

No description provided.

@ControlNet
Copy link
Owner

Thank you for your contribution. I think the changes are too destructive for other tasks now. I will work on the merging.

@aromanusc aromanusc marked this pull request as draft February 9, 2024 06:45
@aromanusc
Copy link
Author

Hey, thanks for the prompt reply. I meant to put this as a draft, I agree the changes are not in the best condition to be merged. I can maybe add a generic dataset loader under /dataset. I also have to remove some loose comments I placed mainly for diff purposes.

I have to say that for the emotion recognition task on the RAVDESS dataset, MARLIN performs great! Now I am working towards an audio-visual implementation with MARLIN. Thanks for keeping this repo in such a great shape.

@ControlNet
Copy link
Owner

Oh my mistake. I thought the pr is for emotion recogntiion for CelebV-HQ. now I have modified the code to make it workable for emotion task on CelebV-HQ.

It will be great if you can prepare the code for RAVDESS with the new python files dataset/ravdess.py, preprocess/ravdess_preprocess.py and preprocess/ravdess_extract.py, and modify the evaluate.py?

@ControlNet ControlNet changed the title enable MARLIN for emotion recognition on the ravdess dataset enable MARLIN for emotion recognition on the ravdess and celebvhq dataset Feb 9, 2024
@ControlNet ControlNet marked this pull request as ready for review February 9, 2024 15:16
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.

3 participants