-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Lens] Missing values - Fitting functions should support start/end values for fitting #72691
Comments
Pinging @elastic/kibana-app (Team:KibanaApp) |
@wylieconlon actually the I'm seeing also a weird artifact on the dashed line |
@markov00 I think it's fine if both start and end values are the same, but I didn't realize that it would depend on the fitting function. Which fitting functions use endValues? |
|
@wylieconlon can we close this issue? It seems like there is nothing to do right now. |
@flash1293 Since all three of the fitting functions that have this behavior are supported in Lens, I think there is a change that should be made. I tested this out locally by setting Before: After: |
@wylieconlon The first behavior seems preferable to me as it's using less assumptions about the data. It's also the behavior of vislib as far as I can tell. It seems like I'm missing the obvious improvement here, can you elaborate? |
The main improvement is that this draws a vertical line, instead of starting the points in the middle of the screen. This would be useful for metrics like uptime/downtime, where the transition between the "no data" and "has data" state is important to visualize. But like all of the fitting functions I think this is use-case dependent, and I'm suggesting that we provide users the choice instead of removing it. |
Makes sense, thanks for clarifying. |
You can see this feature in the charts examples, but Lens does not use the
endValue
setting. As a result, we have charts that have a sudden start value even if the underlying data is intended to be continuous:cc @markov00 I'm not sure that the fitting functions support a
startValue
?The text was updated successfully, but these errors were encountered: