refresh portfolio stats every 11 minutes #65
Conversation
some users would prefer the cex.io portfolio balance should be refreshed periodically with more current BTC balance (as a result of BTC withdraw, deposit, mining new coin with the GHS, etc.) this commit will affect all exchanges, not just cex.io refresh period is 20 minutes
|
Wouldn't it be much better if the interval could be configurable from the config? |
|
@azizmb yes, it would... this was just an initial commit. Not only was the hardcoded 11 in-part because I was being lazy and didn't bother to add config support, but also because using prime numbers instead of "nice round numbers" like 10, 15, or 60 minutes. I try to be friendly to servers for this sort of thing, partially because of my own experience with a POOL.NTP.ORG server I operate --- the load spikes can be rather bad due to the abundance of inconsiderate linux admins who don't consider scalability of their actions when setting up a cronjob to do something like run ntpdate once every night at exactly midnight. |
|
oops, I haven't had my coffee yet and clicked "close" trying to close this browser tab ...and then did it a second time after having had to re-open the pull request / issue once already. |
bugfix for gekko having a stale internal state which doesn't match the actual balance in portfolio
|
see #66 for new version of patch (actually updates the balance before refreshing the display) |
some users would prefer the cex.io portfolio balance should be refreshed periodically with more current BTC balance (as a result of BTC withdraw, deposit, mining new coin with the GHS, etc.)
this commit will affect all exchanges, not just cex.io
refresh period is 11 minutes