-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add Chart #356
Add Chart #356
Conversation
Hi @marina-chibizova! |
@Hjklvfr, are you planning to address the comment, or we can close this pr? |
@evgenydmitriev |
@marina-chibizova @evgenydmitriev Bottom 3 charts are displayed after clicking on one of the top chart |
@Hjklvfr cool! Quick notes from my side:
|
Short answers from my side:
|
I added secrets for codespaces and fix path of script for codespace. You can reload. |
@marina-chibizova Did what I did matches to what needed to be done? |
@Hjklvfr the thing is it doesn't make much sense to visualize every 1m data point separately. Normally people would plot data over time, like several hours or days to be able to see anomalous spikes. Two top charts make more sense to me. Although putting buy-sell ratio and buy-sell ration absolute on one chart is not that beneficial, since they show the same metric, just a bit differently. These things can be fixed though. What's more important is how sensible the widget solution is, and how much maintenance would it require. To asset it I would ask @evgenydmitriev for help. If it's fine we can proceed with visualization fixes |
For now I did it without any additional computations. Just api data. |
As I said, all this can be dealt with after we have a green light for service logic
I didn't quite get what you are saying, /dictionary is very flexible, you can get all the markets at once, or a specific pair with all supported marketvenues or a marketvenue with all supported pairs, or search specific market (unique combination of marketvenue and pair) |
Comments not addressed. Closing. |
resolve #291
Add Chart with market and pair selection
Use React TS