@elasticmachine merge upstream

Pinging @elastic/kibana-app (Team:KibanaApp)

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 0b49124
• Storybooks Preview
• Documentation Changes

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

History

• 💚 Build #111462 succeeded b25057e

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

That's a pretty involved change (I thought it would just be about if (value === null) { value = 0; }.

@nickofthyme do you have a rough time frame for elastic/elastic-charts#1053 ? Will it land in 7.13? If that's the case and it would avoid the need for this PR I would be more comfortable with waiting until it can be fixed in a nice way.

If the upstream fix won't cover the full bug, then it makes sense to merge this.

@flash1293 I'll look at this today and get a better idea of the timeline. My hunch is that this should be easy to fix in charts and patch the 7.12 elastic-charts version.

Per discussion with @dej611 and @markov00...

The gaps are created when values are missing as indended. See https://codesandbox.io/s/voltron-lij5x?file=/src/App.tsx

Green series missing a value at 9:30

which filling missing values with 0would inaccurately indicate continuous data. One option is to set fit to zero for stacked percentage area chart in Lens.

Actions:

• @markov00 to discuss with @monfera tomorrow about a better approach than using fit = 'zero'.
• @dej611 is checking how adding the zero fit option by default would look on the Lens example data.

Closing in favor of #94086