Conversation
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
expected head sha didn’t match current head ref. |
|
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
|
@elasticmachine merge upstream |
|
@elasticmachine merge upstream |
|
@flash1293 I am sorry if my question is silly but I am confused a bit. Here we mention that by default we are hiding the gaps but by default the So this confuses me a bit. Include empty rows feels like filling the gaps to me. |
|
@stratoula its showing at the table level (having rows without a metric value in the table), but hiding at the chart level. Should we change the wording? Or maybe the default? We could do linear interpolation by default maybe. |
|
@flash1293 we changed it to linear by default to our agg-based visualizations. Maybe it makes sense to also do it for Lens? |
stratoula
left a comment
There was a problem hiding this comment.
After Joe's explanation, I tested it locally with many combinations. Code LGTM. It works as expected 👍
|
@stratoula To improve a bit, I changed the copy to |
💚 Build SucceededMetrics [docs]Module Count
Public APIs missing comments
Async chunks
Page load bundle
History
To update your PR or re-run it, just comment with: |


Fixes #117280
Fixes #72691
This PR adds controls for the style of interpolated regions (either use dotted lines or regular style) as well as controls for end values.
This depends on #127453 to work right (otherwise there are no empty buckets to do fitting on)