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

Vega 5 support #31413

Closed
1 of 4 tasks
flash1293 opened this issue Feb 18, 2019 · 9 comments · Fixed by #68639
Closed
1 of 4 tasks

Vega 5 support #31413

flash1293 opened this issue Feb 18, 2019 · 9 comments · Fixed by #68639
Assignees
Labels
enhancement New value added to drive a business result Feature:Vega Vega visualizations Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@flash1293
Copy link
Contributor

flash1293 commented Feb 18, 2019

Vega 5 will be released soon and should be supported by Kibana.

Starting with Vega 4.4, IE 11 isn't supported anymore, which causes Kibana to throw an error on loading. This means the build process has to be configured to correctly compile down the language features used in the code of the vega dependency (and maybe add necessary polyfills for APIs unsupported in IE).

  • Wait for vega 5 to be releases
  • Update vega (switch from vega-lib) and vega-lite
  • Investigate IE 11 errors
  • Extend webpack build config (packages/kbn-babel-preset) to correctly compile everything down

Related Links:

@flash1293 flash1293 added Feature:Vega Vega visualizations enhancement New value added to drive a business result Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Feb 18, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app

@lizozom
Copy link
Contributor

lizozom commented Feb 18, 2019

This tooltip issue will be resolved when this happens.
I will need to clean up the code once it does.

@wylieconlon
Copy link
Contributor

I think this checklist is no longer accurate because vega now builds to ES5 (vega/vega#2179) thanks to @nyurik. The Vega docs page about supporting IE11 seems to be out of date as a result. We need to test the ES5 build to see if will work for us, which is a simpler task than changing our webpack settings.

@flash1293
Copy link
Contributor Author

Wow, that's great! Thank you so much @nyurik . @wylieconlon I'll look into this if you haven't started already.

@nyurik
Copy link
Contributor

nyurik commented Feb 25, 2020

Note that Vega-Lite is still in the process of upgrading -- keep an eye on vega/vega-lite#5920 . I think it is OK for the Vega-Lite to be kept back for a bit until its ES5 situation clears up.

@wylieconlon
Copy link
Contributor

Did some initial work on this. The upgrade seems okay in Chrome, but I can't get it to load in browserstack IE11. If one of you wants to investigate, that would be very helpful: #58545

@wylieconlon
Copy link
Contributor

Progress on vega-lite with ES5 builds, this is now the PR to follow: vega/vega-lite#6280

The remaining steps as I see them:

  1. Wait for vega-lite to release a version with ES5 build
  2. Upgrade to vega 5 and vega-lite 4 together with this PR Vega 5 upgrade #58545
  3. Test in IE11

One thing that wasn't clear to me from the original report was what errors we were commonly seeing in IE11- does anyone have a list?

@wylieconlon
Copy link
Contributor

Okay, so in the last couple hours a new release of vega-lite went out. We are now unblocked from further work.

@wylieconlon
Copy link
Contributor

One of the issues I found with upgrading to Vega 5 is that there are some failures in the Kibana sample Vega visualizations:

Infinite extent for field "y1": [Infinity, -Infinity]

Infinite extent for field "_source.bytes": [Infinity, -Infinity]

Cannot read property 'expressionFunction' of undefined

These failures indicate a bigger issue, and we potentially need to do more work on migrating from old Vega to new Vega.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Vega Vega visualizations Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants