Change copy functions over to use (de)serialization feature

CodingTrain / Toy-Neural-Network-JS

Neural Network JavaScript library for Coding Train tutorials

MIT License

425 stars 242 forks source link

Change copy functions over to use (de)serialization feature #98

Open enginefeeder101 opened 6 years ago

enginefeeder101 commented 6 years ago

The (de)serialzation functions already have all the code needed to make a deep copy of the NeuralNetwork. This commit also adds a test for the Matrix.copy() function. This also copies the learning rate and if #97 is merged the activation function is copied as well.

shiffman commented 6 years ago

If you want to submit the unit tests separately I can merge that now.

enginefeeder101 commented 6 years ago

I pushed PR #99 including just the test so you can merge those.

enginefeeder101 commented 6 years ago

I removed the added tests (see #99) from this branch to allow you to smoothly merge his PR.