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

training/test split and confusion matrix #2

Open
Xiang-Gu opened this issue Jun 16, 2019 · 0 comments
Open

training/test split and confusion matrix #2

Xiang-Gu opened this issue Jun 16, 2019 · 0 comments

Comments

@Xiang-Gu
Copy link

Xiang-Gu commented Jun 16, 2019

Hi, thank you for sharing your code here. I found your code very helpful to me to learn something about FER. But there are two things I think you can improve your code on:
1). The FER2013 dataset (fer2013.csv file) actually comes with training/test labels but in your code, you just randomly select 10% of all the images as your test images. Wouldn't it be nicer to split the dataset as indicated in the csv file?

2). When I run the confmatrix.py to generate the confusion matrix, I get a plot with absolute numbers (like 50, 102, etc) in the cells of the matrix rather than the ratios. Can you modify your code so that it actually computes the ratios and plot it the ratios?

Thanks,
Xiang

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

No branches or pull requests

1 participant