Replace TSVB timeseries charts with elastic-charts#33558
Replace TSVB timeseries charts with elastic-charts#33558alexwizp merged 14 commits intoelastic:masterfrom
Conversation
ed102dd to
6ae639c
Compare
5d48034 to
7904847
Compare
7904847 to
891200b
Compare
💔 Build Failed |
|
retest |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
PhilippBaranovskiy
left a comment
There was a problem hiding this comment.
hey @sulemanof, I guess you might leave conflict marks in the code
💔 Build Failed |
829404c to
197491f
Compare
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
|
retest |
💔 Build Failed |
|
@sulemanof I'm using docker for the 7.3 version compare but cannot reindex the sample data as per the instructions. I get |
There was a problem hiding this comment.
I tested the code locally and compared it with a 7.3 version running in docker, both in Chrome.
Looks great and functions well.
Question:
When did KQL autocomplete stop working in TSVB? It’s not working in 7.3 either.
Confirmation with what's already reported:
For Data -> Options -> Chart type: Bar "Stacked within series" doesn’t work in either version.
Problem I encountered:
In this PR and in 7.3, I could not generate reports

Otherwise, LGTM.
💚 Build Succeeded |
💚 Build Succeeded |
|
Pinging @elastic/kibana-app |
…-series # Conflicts: # test/functional/apps/visualize/_tsvb_time_series.ts
💚 Build Succeeded |
requested change made in 6c17fb8
💚 Build Succeeded |
💚 Build Succeeded |
💚 Build Succeeded |

Resolve elastic/elastic-charts#108 .
Should be checked before merging:
In progress replacements:
Blocker Issues:
/Cannot set the custom formatting function for Tooltip header / Annotation header elastic-charts#197/Fill color and Line width styles are not applied correctly to Bars elastic-charts#323/'tickFormat' property should affect only Axis for which it be declared elastic-charts#185/Disable tooltip on annotation line elastic-charts#324/Dynamic legend content size elastic-charts#311/The value on the legend is not displayed completely elastic-charts#184/The Legend is not displayed correctly in the 'Bottom' mode elastic-charts#194/elasticChartsCrosshair is not positioned correctly for some intervals elastic-charts#195/Cannot set line width (strokeWidth) for BarSeries elastic-charts#196/Color Picker functionality cannot be disabled elastic-charts#198/Percent stacked option elastic-charts#222Tests sample dashboards⚠️
Tasks
Issues 🐛
[eCommerce] Promotion Tracking,[Flights] Delays & Cancellations, also on visualization editor, the visualization doesn't show the crosshair line (usually red)[Flights] Delays & Cancellationsdisable tooltip on annotation line (/Disable tooltip on annotation line elastic-charts#324)[Logs] Response Codes Over Time + Annotationsthe order of the series is inverted, meaning that you have to reorder the way you are adding the series into the Chart component. It's reproducible when using stacked mode on a single visualization with a group by split@AlonaNadler please have a look.
this PR
on 7.3

[Logs] Response Codes Over Time + Annotationsthe annotation header seems always showing the same timestamp. Also reproducible during tests on Annotations[Logs] Response Codes Over Time + AnnotationsI think you are not checking the formatter fornullvalues as in a case you can see theNaNtext hereTest visualization editor⚠️
Tasks
Build a single series visualization
On
Panel optionskibana_sample_data_flightsand choosetimestampas TimestampDrop last bucketshows and hide the last bucket on the seriesOn
Data -> Optionstab of a single series:Chart typeline, barChart typestyles: fill, line width, point size, stepsoffset series,hide in legend,split color theme(only when multi series),separate axis,override index patternusingkibana_sample_data_logsOn
Data -> Metricstab of a single series and with multiple series:Averageaggregation onAvgTicketPriceGroup by: Termsby: OriginData -> optionsoptionsOn
Annotationstabkibana_sample_data_flightsindex pattern ,useDest : "Verona Villafranca Airport" and Cancelled :trueas KQL query, select an Icon, useDestas Fields and{{Dest}}as row template.Issues 🐛
[x] The point size works fine until the value of 10, after that, 11 or more there is no more fill:this is not an issue, but a feature existing on 7.3 also :(
point size: 10

point size: 11 or above

FillandLine widthstyles are not applied to BarsHide in legendseems not working with a single seriesSplit within seriesdoesn't seem to work.Exclamation in circleicon Exclamation mark in circle icon eui#2121Test PDF rendering ✅
Enable trial mode
Start a 30-day trialfrom Settings -> Licence ManagerNew issues fixed by this PR 🎉
before:

now:
0value:on PR
on master/7.3

when using stacked in percentage mode, it automatically apply the
%sign to the data formatwhen using a different timezone than the one in the browser, on the previous version each ticks where rounded to the UTC timezone, losing the nice rounding of the hours/days 00:00, 12:00, 00:00, 12:00
This PR fix that keeping the same rouding

Notes for testing
Compere the visualizations against
masterbranch or at least against 7.3.Using docker with 7.3 is a bit more easy: these are the steps for that:
Write a
docker-compose.yamlfile and run it withdocker-compose up:To use exactly the same dataset (that will allows you to compare also the visualization output use the following from kibana dev tools) this will reindex all the data from the testing elasticsearch into the dockerized ES
Notes for Elastic-Charts 🐛
on 7.3

**on this PR**