Skip to content
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

Major visual changes #37

Closed
wants to merge 3 commits into from
Closed

Major visual changes #37

wants to merge 3 commits into from

Conversation

theimo1221
Copy link
Contributor

The current size was just the root of volume.
In BTC-EUR currently the biggest volume is 14, wich results in invisible bubbles.

I changed it, that now the biggest bubble will always be at size 32.
All other bubble sizes are multiplied with same factor.

It´s tested and looks good.

The current size was just the root of volume.
In BTC-EUR currently the biggest volume is 14, wich results in invisible bubbles.

I changed it, that now the biggest bubble will always be at size 32. 
All other bubble sizes are multiplied with same factor.

It´s tested and looks good.
@theimo1221 theimo1221 changed the title Fixed Bubble Size for low volume pairs Visual changes for low volume pairs Feb 18, 2018
With Logscale the minor range can be viewed more easily.
With Volume filter all sumups less than 1% of Total Volume are filtered. 
Asfaik This is to track Whales
@theimo1221
Copy link
Contributor Author

Works quite good. e.g. BTC-USD:
image

@theimo1221
Copy link
Contributor Author

Additionally this reduces callback load from 18Kb to 5.3 Kb for 8 pairs.
Both measurements taken with #40.

@theimo1221 theimo1221 changed the title Visual changes for low volume pairs Visual changes Feb 18, 2018
@theimo1221 theimo1221 changed the title Visual changes Major visual changes Feb 18, 2018
It uses panda.apply to calculate the correct rgb and adds it as a string
@theimo1221
Copy link
Contributor Author

Now the "bad" boys are highlighted. Brightest color is just 1 order, the darker it gets more unique orders are at that price
image

@theimo1221 theimo1221 mentioned this pull request Feb 18, 2018
3 tasks
theimo1221 added a commit to theimo1221/eth_python_tracker that referenced this pull request Feb 18, 2018
@theimo1221 theimo1221 mentioned this pull request Feb 18, 2018
5 tasks
@theimo1221
Copy link
Contributor Author

Is in #41
@pmaji If you are okay with #41 you can simply close this

@theimo1221
Copy link
Contributor Author

Closed cause of #41 was approved

@theimo1221 theimo1221 closed this Feb 18, 2018
@theimo1221 theimo1221 deleted the patch-19 branch February 19, 2018 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant