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

feat/serialization: add serialization #96

Merged
merged 2 commits into from Apr 19, 2016
Merged

feat/serialization: add serialization #96

merged 2 commits into from Apr 19, 2016

Conversation

hobofan
Copy link
Member

@hobofan hobofan commented Apr 6, 2016

WIP, the commit for deserialization is still missing because it needs a bit of polish

Will resolve #14, #15

@hobofan hobofan force-pushed the feat/serialization branch 8 times, most recently from 31812f8 to 47f8d46 Compare April 12, 2016 21:51
@hobofan hobofan changed the title [WIP] feat/serialization: add serialization feat/serialization: add serialization Apr 12, 2016
@hobofan
Copy link
Member Author

hobofan commented Apr 12, 2016

@MichaelHirn :)

@MichaelHirn
Copy link
Member

Thank you, @hobofan

Are now all layers serializable? Also could you add a test/example of the serializing/deserializing a network?

@hobofan
Copy link
Member Author

hobofan commented Apr 17, 2016

@MichaelHirn All layers are serializable, and I added a basic test of saving and loading, and a example for each.

@MichaelHirn
Copy link
Member

LGTM. Thank you for the update. Ready to merge

@MichaelHirn
Copy link
Member

@homu r+

@homu
Copy link
Collaborator

homu commented Apr 19, 2016

📌 Commit df7c9d8 has been approved by MichaelHirn

@homu
Copy link
Collaborator

homu commented Apr 19, 2016

⚡ Test exempted - status

@homu homu merged commit df7c9d8 into master Apr 19, 2016
homu added a commit that referenced this pull request Apr 19, 2016
feat/serialization: add serialization

WIP, the commit for deserialization is still missing because it needs a bit of polish

Will resolve #14, #15
@MichaelHirn MichaelHirn deleted the feat/serialization branch April 21, 2016 13:34
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.

load network with weights from file
3 participants