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

Chart Y axe spikes #56

Closed
EthraZa opened this issue Jul 24, 2024 · 3 comments
Closed

Chart Y axe spikes #56

EthraZa opened this issue Jul 24, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@EthraZa
Copy link

EthraZa commented Jul 24, 2024

When the ordinary response time is about 20 to 100 or even 200ms, the chart is still ok, but when it spikes to 3000ms+ in the Y axe, the rest of X axe becomes just a line with small ondulations.

Captura de tela de 2024-07-24 11-59-44

What can be done...
As you can see in this example, Chart.js has a logarithmic type that could "fix" that spikes on chart.

download

But, since the react-chartjs-2 looks like is abandoned and the docs are not working, I can't figure out how to apply this option to chart using it, if even possible.

@lyc8503 lyc8503 added the enhancement New feature or request label Jul 24, 2024
@lyc8503
Copy link
Owner

lyc8503 commented Jul 24, 2024

Personally, I think such spikes do represent timeouts or access delays occurring during probing, and we don't really need to use logarithmic units here, logarithmic units may rather confuse others. (Though maybe adding a separate config item to enable logarithmic units would also be ok, I'm not really sure)

If your service latency is generally very low, you could try lowering the timeout in your configuration, which would prevent very high spikes from affecting the chart.

@EthraZa
Copy link
Author

EthraZa commented Jul 24, 2024

If I set a low timeout it would show as a fail when it could respond, don't?
It could be an option for each monitor to enable logarithmic, maybe.

@lyc8503
Copy link
Owner

lyc8503 commented Aug 21, 2024

Sorry, I still don't think it makes sense to use logarithmic axes for time. This seems rather misleading, so I wouldn't make it an official option. You can modify the chart in components/DetailChart.tsx in your own deployment if you want.

As for the documentation for react-chartjs-2, someone has deployed an unofficial version of it themselves, so you can check that out. reactchartjs/react-chartjs-2#1219 (comment)

For anyone else reading this, I'd still recommend lowering down the timeout, if your service does tend to respond quickly.

@lyc8503 lyc8503 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants