search

iamtrask / Sonar

DEPRECATED - Decentralized Machine Learning Server (hosted on Blockchain)
Apache License 2.0
141 stars 49 forks source link

Larger account numbers #17

Closed carrollgt91 closed 7 years ago

carrollgt91 commented 7 years ago

This PR assumes https://github.com/OpenMined/Sonar/pull/16 will also be merged, though this change could easily be made separately.

I bumped the number of accounts in this image to be compatible with the PySonar demo and make it easier for mine.js to utilize this image, as to see the full demo of mine.js you must currently use the PySonar demo.

anoff commented 7 years ago

Discussed this issue with @iamtrask a while ago. Keeping the account number small speeds up the startup time of testrpc significantly (1s vs 6s). I don't see a reason to have such high account numbers besides the (legacy) notebook.

Then again 42 might be a stupid number, bumping it up to 100 might be easier: 100 accounts with 100ETH each.