-
Notifications
You must be signed in to change notification settings - Fork 73
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
[Help Wanted] Is CifarCryptoNets Functional? Low Accuracy with Small Test #1
Comments
We are working on this issue, sorry about the delay |
Me too!
By the way, It REALLY takes long for cifar10.. |
We are working on resolving this issue. Sorry it is taking so long. |
I just pushed a fix to this issue. The main problem was DataPreprocessing but during the fix we generated a new model for CIFAR that has higher accuracy (76.31%) and runs in 750 seconds. Please check the readme.md for small changes in the command line arguments. We also started documenting the process of converting a model trained with Keras to work with this package. |
I was able to reproduce the output provided for CifarCryptoNets. I believe they are the prediction values for the 10 categories in Cifar-10 and that the max value is the prediction.
I tested the model separately with the full Cifar-10 dataset and with 1 test image from the dataset and produced the same output which, I believe, is because the code doesn't loop through the network with the GetNext() method like the other provided models.
After iterating through 10 test images and took the max value as the prediction as displayed below, I noticed that the accuracy based on my approach is roughly around 10% instead of the higher accuracy expected of LoLaCryptoNet.
Is the CifarCryptoNet fully functional? Please advise.
The text was updated successfully, but these errors were encountered: