-
Notifications
You must be signed in to change notification settings - Fork 10
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
Regression with CNNs ? #69
Comments
Current scores: | Id | Param | Type (1=RNN, 0=CNN) | Status | |
It seems it has been a while that I did not train on CNN. I leave that open for the future, when comparing with the models we have for Old French. |
Hey, I believe bca9516 fixed some previous hacks with the CNNs, it's an old commit I had lying around. I am surprised I hadn't pushed it before. I haven't used cnn embeddings very extensively, so I am cannot rule out a bug for sure. It would need some experimenting to make clear what's going on. Same architecture trained with previous and current implementation and comparison. |
Hey :)
Weird thing: as I actually had equivalent scores between CNN and RNN embeddings, my experiments with Optuna for now have yielded consistently lower result (by a big margin) on CNNs (I assume this change is in cause bca9516#diff-4853226f237bf1b0a17a302fbfa3e997 )
I cannot say if it's a regression before I actually retrain the same config as before but this might be something to look at in the next weeks.
The text was updated successfully, but these errors were encountered: