search

zack-bitcoin / amoveo

A blockchain for trust-free markets in financial derivatives
Other
466 stars 110 forks source link

Miner.js speed calc #96

Closed Mandelhoff closed 6 years ago

Mandelhoff commented 6 years ago

https://github.com/zack-bitcoin/amoveo/blob/6781b16a36fcafb1072d4a55c27e9e45194a8a76/apps/ae_http/priv/external_web/miner.js#L45

Should the miner.js speed calc be "1000 / d" instead of "10000 / d"?

I'm not 100% sure, so I didn't open a pull request for that change.

zack-bitcoin commented 6 years ago

You are correct. Thank you for noticing. I rewrote it to use more variables and better variable names, I will push the changes after testing.

zack-bitcoin commented 6 years ago

I pushed the fix. I haven't yet updated the server for this change.

zack-bitcoin commented 6 years ago

ok, now I updated the server.