-
Notifications
You must be signed in to change notification settings - Fork 251
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
support stats using Block/BlockStat table #107
Conversation
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.
ACK
769204d
to
d18d571
Compare
Epic commit! Will update https://classic.chainkorea.kr when we merge this one 😄 |
Excellent work! Here's my initial comments:
|
yes. l have already made some fix, push it later soon.
nope. anyway it is easy to make it configurable in
correct. |
|
* plugins/moment/*.js added * FIXME The bome chart does not work correctly
* aggregate() with proper match options * verify range/days req params * fixed tools/stats.js to support rescan with a custom interval. * show percent at a tooltip
* hide low rank miners * adjust height for mobile
@hackmod show high rank miners first. hide low rank miners. - This part was pretty annoying until it solved now 😄 |
WARNING
this is a experimental fix to serve statistics without external apis (http://drawpie.com/etc_hash_rate_api is currently broken)
Please do not merge it, execpt for testing.
public/js/stats/*.js
javascripts into angular directives. (almost no change except external apis)getMinerStats()
using theBlock
table (instead of BlockStat table)getHashrates()
using theBlockStat
table.The bomb chart is not work correctly.fixed.prepare & run
node tools/stats.js
: initializeBlockStat
table to show the network hashrate chartRESCAN=10000:1000000 node tools/stats.js
(interval:steps
)npm start
screenshot