-
Notifications
You must be signed in to change notification settings - Fork 43
Knowledge distillation, fix and improve cross-entropy #229
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
Merged
Merged
Changes from all commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
5137757
stuff
jlamypoirier f0cb32a
Merge remote-tracking branch 'origin/main' into config_updates
jlamypoirier f26010e
Update pretrained config
jlamypoirier b930a39
stuff
jlamypoirier 918a7a8
Merge branch 'config_updates' into update_pretrained_config
jlamypoirier 8117c47
fixes
jlamypoirier 1c995d3
fix
jlamypoirier 3f90475
Merge branch 'main' into config_updates
jlamypoirier e389058
Merge branch 'config_updates' into update_pretrained_config
jlamypoirier 506fe92
fixes
jlamypoirier 971d3ef
fixes
jlamypoirier 6bf20cb
Tests wip
jlamypoirier c13fb19
misc
jlamypoirier a20fcec
tests
jlamypoirier 9af26a7
Merge branch 'main' into config_updates
jlamypoirier 9af372d
Tests, fixes, remove tuple format
jlamypoirier dded00a
fix
jlamypoirier 42d5ca4
Merge remote-tracking branch 'origin/main' into config_updates
jlamypoirier 986f9f3
fix
jlamypoirier 5abc087
Merge branch 'config_updates' into update_pretrained_config
jlamypoirier 8e3e795
fixes
jlamypoirier da6eb7b
fixes
jlamypoirier 67e08aa
Merge branch 'main' into config_updates
jlamypoirier a09e6f3
Merge branch 'config_updates' into update_pretrained_config
jlamypoirier baad705
fix
jlamypoirier b702837
Test, fixes
jlamypoirier a8684f8
Knowledge distillation, fix cross-entropy
jlamypoirier b781729
Fixes, distillation
jlamypoirier db6504b
fixes
jlamypoirier 7c2933a
Merge remote-tracking branch 'origin/main' into config_updates
jlamypoirier a017c11
Merge branch 'config_updates' into update_pretrained_config
jlamypoirier 368a6bf
Merge remote-tracking branch 'origin/main' into update_pretrained_config
jlamypoirier e0c82a0
Merge remote-tracking branch 'origin/main' into distillation
jlamypoirier 16a3dd7
Merge branch 'update_pretrained_config' into distillation
jlamypoirier cff9892
fixes
jlamypoirier 793ecde
Merge branch 'update_pretrained_config' into distillation
jlamypoirier b67006a
fixes
jlamypoirier 2014108
Add constraints
jlamypoirier 4fb78e4
Merge remote-tracking branch 'origin/main' into distillation
jlamypoirier fa3d556
Add constraints
jlamypoirier 48141e5
Merge remote-tracking branch 'origin/main' into update_pretrained_config
jlamypoirier e6e5a32
Merge branch 'update_pretrained_config' into distillation
jlamypoirier 537deca
fix
jlamypoirier 3d5dc94
Merge commit '6ad0a96c9328234b907d01a82c4c52bd48752b2f' into update_pβ¦
jlamypoirier 2bb0c08
Merge branch 'update_pretrained_config' into distillation
jlamypoirier 067ba97
Merge remote-tracking branch 'origin/main' into distillation
jlamypoirier d2b3154
misc
jlamypoirier File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bit of a hack, no?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kind of, adapting the current setattr to deal with method override would be very difficult so I'm bypassing it. (And that's how frozen dataclasses do it)