Hosting about crypto-whale-watching-app HOT 55 CLOSED

Current Situation: Once we loose connection to Gdax our refresh thread terminates on error, while server keeps running, cause he is on main thread.

Solution in #52: Own thread for both Server and Refresher. Main program acts as a watchdog and if it detects a dead thread it restarts the thread.

I tested this by unplugging ethernet cable, (wich causes exception due to gdax pull), and watchdog restarted thread fine. Site get´s new data!

So @pmaji please accept pull and @CrackLord please update Server.
I´m sorry for not having this thought before, but both of you getting stuck at excact same time, gave me this idea!

from crypto-whale-watching-app.

Done, @pmaji

from crypto-whale-watching-app.

Added some stuff in #48
We had following problem:
He was updating 8 pairs one after another.
Then waits 5s and again pulling 8 pairs.

This is problematic with the Api due to many request at short timeframe.

I changed it to 1 request 1s pause 1 request ....

from crypto-whale-watching-app.

@CrackLord #54 was merged 5 hours before my post.
You are using an outdated version ;)

I have updated now again to be sure it's on the latest version.

from crypto-whale-watching-app.

Hi guys, sorry for the lack of responses, I've been busy. I have spent this morning getting you setup on your own VPS on a separate account on https://digitalocean.com. Unfortunately my VPS with Ramnode kept getting disabled within the last few days of the month every month because it was hitting the bandwidth quota. It's now using the $5 per month VPS on DigitalOcean on an account that I can give you access to. There is already $20 worth of credit on there which will allow us to run it for 4 months.

With DigitalOcean's UI we'll be able to monitor the bandwidth usage more easily and see what's appropriate. I'm still running an old version of the app for now as I've not been following development.

I've also moved you onto a new domain at https://whales.crypto.app/ with HTTPS enabled. This is only temporary while you find a domain for your app as I cannot guarantee continued ownership of this domain forever. The old domain still works and just redirects to the new one.

from crypto-whale-watching-app.

FYI we are stilling dealing with a problem related to the present hosting. After a short while of working without a problem, the app ceases updating with new prices.
@CrackLord (the webhost) and I don't believe it has anything to do with throttling from GDAX because we are well within the limits they list on their website.
@CrackLord could you perhaps give a brief description of how you are hosting it presently so that others might guess as to the issue. My thought is that @theimo1221 might have a hypothesis.

from crypto-whale-watching-app.

@CrackLord could you perhaps give a brief description of how you are hosting it presently so that others might guess as to the issue.

It is running on a VPS. It is simply being run by running pyhthon app.py using a systemd service and then reverse proxying requests to the app server using nginx.

It is a simple setup, I don't think that this would be related to the issue.

from crypto-whale-watching-app.

I guess the problem is within the sheer amount of requests made by the website, after a while servers start refusing requests.
Changes made in #40 should definitly change this.

@CrackLord could you update the version on your server after @pmaji approved the changes?

Thanks and Greets

from crypto-whale-watching-app.

@CrackLord could you update the version on your server after @pmaji approved the changes?

Will do. Error logging on the GDAX API endpoint would be useful too, to see what's going on. The logs are somewhat full of crap at the moment though because of the default http Python server logging so it would be worth disabling that I think.

from crypto-whale-watching-app.

My changes are combined in #41

@CrackLord for testing stability you could just copy my code from the link above. It includes all changes I made today.

I´ll take a look into logging the endpoint but I guess thats not the problem

from crypto-whale-watching-app.

@CrackLord you are good to integrate the new version from Master and start hosting. Thanks to @theimo1221's improvements, this should be sustainable and avert the refreshing-stopping problem previously experienced.

from crypto-whale-watching-app.

FYI @CrackLord @theimo1221 the new version appears to already have de-synced. Back to the drawing board I suppose.

from crypto-whale-watching-app.

I just took a look into it.

It´s not current data, but the request to the server and the response seem to be totally okay.

So we desync but on App side, not on client side.

So it could be something in @CrackLord settings or GDAX side, blocking the data....

I think I´ll add a time to the data to be able to see the last correct refresh from gdax

from crypto-whale-watching-app.

I just had ideas what might be the problem and it could be very easy:

1. He uses a VPS. Others from the same hosting provider on the exact same hardware might as-well be calling data from gdax resulting in api Errors.
2. We call the Api multiple times. It would be better to call it once and cache data for other pairs.

Will fix this

from crypto-whale-watching-app.

@theimo1221 great catch. I glossed over that given our previous change to 1 data pull and neglected to realize it might still be treated by GDAX as one after the other leading to throttling. Let's see if this one works to solve it.

from crypto-whale-watching-app.

@CrackLord have you updated version on your server?

from crypto-whale-watching-app.

from crypto-whale-watching-app.

Okay so I ran a local instance of the app as well for testing purpose and both @CrackLord 's version and my own stopped refreshing at 19:17.

I started my version at 17:55:44 (similar to when CL started his).

@theimo1221 Any ideas on what is causing the refreshes to stop over an hour after the initializing?

My one thought presently would be we could push the wait time a bit further back and retest.

from crypto-whale-watching-app.

@pmaji Cracklords stopped at 19:17 aswell.
But I can still connect to it and get the data.
So in my opinion it can´t be a critical error (Critical error would lead to end of main program, wich terminates server thread).
We should get some debugging from gdax. Maybe we can catch the error from Gdax and prevent it by an one time additional waiting....

from crypto-whale-watching-app.

Wait it´s the other way around, refreshing is a thread, while Server isn´t guess I spot the problem.

from crypto-whale-watching-app.

@CrackLord please update your site again

from crypto-whale-watching-app.

@CrackLord please update your site again

I have already been #52 since it was merged :)

from crypto-whale-watching-app.

@CrackLord #54 was merged 5 hours before my post.
You are using an outdated version ;)

from crypto-whale-watching-app.

@CrackLord go ahead and re-pull. You'll also need to download the two new files (the new .py [and maybe the .js if you want as well]) if you haven't already. Let me know whenever it's done :)

from crypto-whale-watching-app.

Done @pmaji

from crypto-whale-watching-app.

Ty @CrackLord, but is your server down? Can´t reach it

from crypto-whale-watching-app.

There seems to be a problem with the latest version @theimo1221. It doesn't work for me locally either.

Edit: I've rolled back to 623d21d until there is a fix.

from crypto-whale-watching-app.

Hm, it was both for @pmaji and me. Did you download gdax_book.py?
He´ll approve the newest pull today let´s see how that one is running for you

from crypto-whale-watching-app.

@CrackLord Talked with @pmaji and newest Pull with Sidebar is working completely fine on our machines

from crypto-whale-watching-app.

I don't see any graphs when using the latest version at all. I'm not sure why this would work for you but not for me.

from crypto-whale-watching-app.

@CrackLord Wait a little bit or did you forgot to download gdax_book.py?

The process now needs like 40s before data are ready
Edit: To be precise:
11 * 4s for websocket starting
11 * 3s for data calculation
So after 80 seconds all should be fine 😉

from crypto-whale-watching-app.

@CrackLord Wait a little bit
The process now needs like 40s before data are ready

I see what the issue is. Your latest changes are suddenly very resource intensive. It's using 100% of one core of my CPU on my Macbook and now taking up 256 MB of RAM. This is causing the app to crash on the server, since the server is not as powerful as my laptop.

Edit: I believe the intensive CPU usage is caused by this. This is essentially using up the entire core the script runs on to the max limit because there is no throttling on the loop.

from crypto-whale-watching-app.

@CrackLord on my pc it takes like 14% CPU

You can add a sleep there sure, but others are not sleeping aswell, due to the fact, we can constantly pull by having Websockets.
But I´ll maybe rewrite Code to let Calc wait on successfull prepare first.

from crypto-whale-watching-app.

@CrackLord on my pc it takes like 14% CPU

Yes but how many cores does your CPU have? That is probably close to 100% of one core.

Listening and updating on a websocket is fine but a while loop without a throttle will naturally max out the CPU core it's running on. Without a throttle then it will run as fast as the CPU allows it, which means maxing out the resources it's given.

from crypto-whale-watching-app.

@CrackLord
I understand your point but to serve live data we have to be as fast as possible.

I'll rewrite the calc and preparation to wait for new data which will result in sleeping time and even up updating cause we start calculation at correct time

from crypto-whale-watching-app.

I understand your point but to serve live data we have to be as fast as possible.

Yes but it doesn't make sense to do that in a while loop, it should be done as new data comes in for that pair.

Also imo the calculations should be done on the frontend using JS, not on the backend. It makes sense to make the client do all of that work. Streaming the data to the client via websocket.

from crypto-whale-watching-app.

Will do that tomorrow

For now I guess it'll be fine if you add some sleeps on your side

from crypto-whale-watching-app.

Feel free to give the sleeps a try on your end @CrackLord and let us know how it goes. If it's still overloading you can revert to a slightly older version of the code until we chill out the CPU usage :)

from crypto-whale-watching-app.

I did some logging on this:

As you can see the data sequence difference for some pairs are much higher due to computation time consumption.
Resulting in differences of total refreshes, but faster refreshes for pairs with less data.

But it´s not like 1 sequence every x ms, so i tried it with a global var set to 30s for each Pairs refresh.

Now all have same amount of total refresh and CPU load is less.

from crypto-whale-watching-app.

@CrackLord if you update don´t forget to take #82 with it, cause it fixes a bug wich can lead to constant restarting of a websocket

from crypto-whale-watching-app.

@CrackLord if you update don´t forget to take #82 with it, cause it fixes a bug wich can lead to constant restarting of a websocket

I've updated to master, including the change from #82 but it's still killing the server. It's possible that it may be memory usage, the server I'm using only has 256 MB of RAM and it's using all of that and then starts going into swap memory.

If you guys want @pmaji, I can get us set up on a DigitalOcean VPS, the smallest one is much more powerful than my spare VPS and there's a lot of room to scale up if necessary to extremely overpowered servers. I messaged @pmaji on Reddit about this already. Once I get it set up there, it should be relatively maintenance free and we can also enable automatic backups of the server so it can be restored with the click of a button if anything goes wrong.

The current amount of donations should be able to sustain hosting, SSL and a domain for at least a year, if not more.

At the very least I'll get it set up and it's only pay by hour, so if it ends up not working out we don't lose any investment really. What do you think about that @pmaji ?

from crypto-whale-watching-app.

@CrackLord I think it's definitely viable. At a minimum I'd like to test it to see what kind of a baseline cost estimate we get for just one hour. Feel free to do that but don't take on anything larger immediately. Still have to discuss performance differentials with @theimo1221 before deciding.

from crypto-whale-watching-app.

@CrackLord I think it's definitely viable. At a minimum I'd like to test it to see what kind of a baseline cost estimate we get for just one hour. Feel free to do that but don't take on anything larger immediately. Still have to discuss performance differentials with @theimo1221 before deciding.

The costs are very predictable. They would essentially be $0.007/hr if the lowest specced VPS is sufficient, which I believe it would be.

from crypto-whale-watching-app.

In the following the description I send @pmaji

Old version:

1. Pair 1
   1.1 get data from gdax for pair
   1.2 calc graph
   1.3 recalc send data
2. Pair 2
   2.1 ....
   2.2

So before pair 1 refreshes all other pairs have to be refreshed
And each takes minimum 500ms to prevent gdax limitation and for some pairs calculation takes 3-5 s

New version:

Thread 1: Server
Thread 2: sum up data of cache
Thread 3-14: websocket for each pair
Thread 15-26: pull from websocket and calc of data for each pair
Thread 27-38: prepare of graph data

All those are running parallel
So if pull/calc + prepare just needs 5s than that pair is updated every 5s
If it takes 20s than this pair is refreshed every 20s while others keep refreshing at their speed
Of course having more threads and load will slow down each thread if cpu can't take more load but until that point is reached it's a huge plus

I hope this gives a good understanding of difference between both methologies

And to be scaleable in future we have to use new method

Imagine we have 100 pairs average needing 3s to refresh and calculate data. With old method this would result in each pair having new data every 5 minutes

With new method 100 pairs (resulting in 302 Threads)
Would result in cpu load wich slows all down but still it will be around 30-45s for slowest pairs and many would be faster. Resulting in kind of live data

from crypto-whale-watching-app.

Yep, I think that's well understood. It seems to me that this is the way we want to go, as long as it doesn't wind up being cost prohibitive.

@theimo1221 could you double check by looking over master to be sure that this is the most up to date version of the threading methodology that is optimized to strike the right balance of speed and CPU load?

@CrackLord; upon confirmation, feel free to test the new Master for the shortest time period possible (I think an hour). That can give us a baseline of price expectations. I'll shoot you ETH to pay for whatever it winds up being; just dm me the invoice. I'll need to build into my forecast thereafter what the price change will be given user load, so if the cite gives an estimate of that I'd appreciate it as well.

from crypto-whale-watching-app.

@pmaji Please pull #83, which changes Js link back to cdn version wich is updated. (No need to use the developer link, this could even lead to an error, if too many use it.)

@CrackLord Experiment with following line.
desiredPairRefresh = 30000 # in ms

The lower it is, the better is it regarding speed of at least some pairs, the higher it is, the less cpu load it takes.

from crypto-whale-watching-app.

@theimo1221 @CrackLord pulled. Feel free to do some testing on your end @CrackLord to see what is optimal.

from crypto-whale-watching-app.

@CrackLord What is current state?
@pmaji It´s now 10 days since my updates, wanna see them online!^^

from crypto-whale-watching-app.

@pmaji What is the current state here?

from crypto-whale-watching-app.

Hi guys, sorry I've been particularly busy these last few weeks. I've also spent some time thinking about this project and how it could be improved. My following suggestion is going to be pretty drastic but I think it'll be the ultimate way to create this app.

I think you should move away from using Python or having any backend and only use client-sided JavaScript. This solves a lot of issues.

First issue is performance, all of the calculations, requests/websocket listening will be done on the client-side directly with GDAX, so all of the hard work is put entirely on the client.

Second issue is hosting, hosting static files requires literally near 0 cost. You can host on Github with SSL from CloudFlare for free, all you need to pay for is a domain which is $9,99 a year for a .com domain at most.

Third issue is making it look good, with the current Python libraries and architecture being used, it's quite difficult to edit the HTML to make it look good and add custom stylesheets etc. If this were simple HTML + JS file(s) then it would be a lot easier to manage the look and feel of the site.

That's my 2 cents guys. I know that this would be a pretty drastic change but the most important part of this code is really the algorithm which determines which orders are from the same user. That part can easily be ported to a new JS codebase.

I really do think that this would be the ideal way to go forward and don't see a reason not to do it, other than the fact that it would throw out a lot of the existing code.

from crypto-whale-watching-app.

@CrackLord I understand and appreciate your points, but disagree on 2 points:

Design

1. iFrame
   Putting our content in an iFrame gives us all possibilities Html/ JS/ Css offers
2. change content with js.
   As already in newest version I add styles and content with JS on clientside

Performance:

Having all Api´s and calculations on clientside is not useable depending on internet connection and performance, especially on mobile devices.
And if me add more exchanges, some doesn´t offer web api, or atleast limit it

from crypto-whale-watching-app.

@CrackLord I've given it some more thought as well, and while I appreciate the suggestion, I don't think that's viable. I know @theimo1221 outlined some reasons why, but I'll summarize what I think are the most important ones:

1 - Performance. Thiemo called this one out well--with the present thread-styled update regime, it would make the app not usable for some, and the usability problem would only get worse as we look to adding more exchanges.
2 - Long term, there may be many things we want to do (like new exchange addition, as Thiemo called out) that don't offer full service web APIs, or that throttle us. Also, throttling limits are likely to change and get worse as time goes on, meaning we need back-up plans.
3 - This is probably the biggest one. I'm simply not comfortable in JS. I can proof-read JS code and understand it, but re-writing the entire codebase in JS isn't something I want or have time / expertise to do. I don't want to lose the Python-centricity of this project long-term.

If where that leaves us is that you just want to keep hosting the older version of the codebase without the speedy (but slightly more comp-intensive) updates, then I completely understand. We'll keep pressing on and hopefully find a cost effective way to host newer improvements.

from crypto-whale-watching-app.

@CrackLord — do you have an eta on when the site will be back up? We’re doing a video on the app shortly and wanted to be sure the web version was live.

from crypto-whale-watching-app.

@CrackLord thanks for the update! This is great news. I'd be interested to see how the newest live version performs on the new VPS and if it significantly increases the cost.
Also, could you pm me your Telegram name via reddit? I'll add you to the dev chat--probably easier than bogging down this thread with a back-and-forth--so we can iron out any details. Ideally I'll like to be able to take as much off your hands as possible.

from crypto-whale-watching-app.

Closing this out given the more recently active other issue on this subject, which you can find here if interested.

from crypto-whale-watching-app.