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

Alex's homework #24

Merged
merged 1 commit into from
Mar 20, 2017
Merged

Alex's homework #24

merged 1 commit into from
Mar 20, 2017

Conversation

apozas
Copy link
Collaborator

@apozas apozas commented Mar 20, 2017

Code (APS_Cracker.ipynb) + trained weights (APS_Cracker_weights.h5)

Code (APS_Cracker.ipynb) + trained weights (APS_Cracker_weights.h5)
@peterwittek peterwittek merged commit 2a797ba into peterwittek:master Mar 20, 2017
@peterwittek
Copy link
Owner

Wow, 97% accuracy on the real-world data with data augmentation, and with only a few minutes of training! This is awesome. On the other hand, adding the h5 file was not a great idea: it is a generated file, which by principle of version control systems, should not be there, plus it is a 90MByte binary that now will be there forever (deleting is not an option since git will remember it anyway).

@silky
Copy link

silky commented Mar 20, 2017

@peterwittek - i'm not sure if it can be used after the fact, but maybe check out - https://git-lfs.github.com/

@peterwittek
Copy link
Owner

I was trying bfg, but did not get very far.

@peterwittek
Copy link
Owner

Okay, I think it is resolved now. Both the binary blob and its history vanished. @apozas, please delete all local clones as well as your fork. Then fork it again and clone. The history of your current fork is inconsistent with the altered history of the fixed-up upstream version.

@apozas
Copy link
Collaborator Author

apozas commented Mar 21, 2017 via email

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