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

[BUG] Crosshair value misreported on the right axis #291

Open
4 tasks done
adeacetis opened this issue Mar 14, 2023 · 6 comments
Open
4 tasks done

[BUG] Crosshair value misreported on the right axis #291

adeacetis opened this issue Mar 14, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@adeacetis
Copy link
Collaborator

adeacetis commented Mar 14, 2023

  • I added my browser's console output in the issue (open the console first, try to reproduce the issue then see if there is anything)
  • I have attached an export of my aggr database in the issue
  • I documented the issue with screenshots
  • I provided a clear and concise description of what the bug is (see both sections below)

Describe the issue

When I use the SOL chart 50 ticks with 5 connections. I have noticed that the crosshair value doens't report accurately on the right axis.

This is issue probably occurs because we are displaying index price and not the actual feeds.

I would expect the value to match the chart I have in front of me even, if it's not actionable trading wise.

Besides, I also note that the right axis isn't showing values accurately. See below with repetition of 21.5 level.

aggr-ocfr.txt

image

image

image

...

Steps to reproduce

(Preferably from a clean environment / incognito mode)

  1. Setup SOLUSD connection as shown above,
  2. select 50ticks,
  3. wait awhile that price actually fill a bit,
  4. then hover the chart and look at the value reported on the right axis.
    ...
@Tucsky
Copy link
Owner

Tucsky commented Mar 14, 2023

Looks like the crosshair is showing the right value.
The bug is the repeating 21.5, 22.5 etc
a weird case for sure

@adeacetis adeacetis added the bug Something isn't working label Mar 14, 2023
@adeacetis
Copy link
Collaborator Author

adeacetis commented Mar 14, 2023

I have had another case before it where the scale was right but not the crosshair.

@Tucsky
Copy link
Owner

Tucsky commented Mar 14, 2023

I have had another case before it where the scale was right but not the crosshair.

with tick chart also?
I can't reproduce the bug right now

image

@adeacetis
Copy link
Collaborator Author

I can't reproduce it as well. I have opened a new chart and waiting for it to fill in to see if this occurs again.

The bug shows that the scale couldn't display two decimals. Now, I have two decimals, like everything is normal.

@adeacetis
Copy link
Collaborator Author

adeacetis commented Mar 14, 2023

If it fixes itself with a simple page refresh, it probably is a wont fix for now until we figure out how to reproduce those steps.

However, as tick chart doesn't have historical data. That could be quite annoying for some users if this is related to this data type.

@Tucsky
Copy link
Owner

Tucsky commented Mar 15, 2023

I never use these tick charts, aggr purpose is to aggregate multiple feeds into one chart and it doesn't mix well with tick chart

quantower makes this definition for tick chart
image

that's too bad because for the largest crypto exchange out aggr (client side) doesn't use raw feeds for trade, meaning tick chart on that exchange just plain wrong
https://github.com/Tucsky/aggr/blob/master/src/worker/exchanges/binance_futures.ts#L76

At the moment the aggregator only know a list of market to connect to, each panes have one, we just merge all lists and connect to every market the same way : I was working on a feature dedicated to get the data type required by each panes respectively allowing one pane to get the raw feeds, another to get only ticker feed, or just normal trade feed or even orderbook feed. (branch feature/channels, huge refacto)
Yet another feature I didn't finish cause it got too big lol, I should get back on that branch and only get the channels part...

TLDR aggr tick charts useless til feature/channels is integrated 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants