keplergl / kepler.gl

Kepler.gl is a powerful open source geospatial analysis tool for large-scale data sets.
http://kepler.gl
MIT License
10.39k stars 1.74k forks source link

[Bug] Error when setting Grid Layer height property #1467

Open openlayers-droid opened 3 years ago

openlayers-droid commented 3 years ago

Describe the bug An error in deck.gl: Cannot read property '3' of undefined in lirko2f when choosing Grid layer

To Reproduce Steps to reproduce the behavior:

  1. Go to 'import csv'
  2. Click on 'import'
  3. choose Grid layer
  4. toggle Height
  5. choose 'Height based on'
  6. enter field - int range
  7. a red notification box 'An error in deck.gl: Cannot read property '3' of undefined in lirko2f'

Expected behavior display grid as in on keplergl.com/demo

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Additional context Using the same dataset, it works as expected on https://kepler.gl/demo

macrigiuseppe commented 3 years ago

@openlayers-droid can you please provide a sample of your dataset (csv file)?

openlayers-droid commented 3 years ago

hi @macrigiuseppe

many thanks for the response. What i found was the same file works on the hosted version. But when i download and build locally it behaves like above. i will create a subset of the csv file and attach it here.

openlayers-droid commented 3 years ago

@macrigiuseppe here's a video link: https://www.youtube.com/watch?v=xwvOxIzeIiA

ibgreen commented 3 years ago

@igorDykhta This may be related to one of the issues you are working on?

igorDykhta commented 3 years ago

@ibgreen haven't encountered this crash

openlayers-droid commented 3 years ago

@igorDykhta it happens only on localhost

ibgreen commented 3 years ago

@igorDykhta of course. What I meant is that you are about to overhaul the height handling of a number of layers in deck.gl.