Kepler.gl: Deck.gl Versioning Issue

Created on 13 Aug 2019  路  5Comments  路  Source: keplergl/kepler.gl

Describe the bug
The deck.gl version is set to ^7.1.5 in the package.json, and the lockfile specifies 7.1.5. So, when cloning and running this repository as-is, everything works properly. However, when using yarn add kepler.gl in an external project, [email protected] is installed. This leads to the following error upon attempting to load data (occurs within deck.gl):

Uncaught TypeError: Cannot assign to read only property 'mousePosition' of object '[object Object]'

To Reproduce
This can be simulated in this repository by removing the lockfile:

  1. Clone repo, navigate into directory
  2. rm yarn.lock
  3. yarn --ignore-engines
  4. yarn start
  5. Go to web browser, attempt to load a dataset
  6. See error in console

Expected behavior
package.json should target only compatible versions of deck.gl, allowing for yarn add to external projects.

Desktop (please complete the following information):

  • OS: macOS Mojave
  • Browser: N/A
  • Version: 1.1.3
  • Yarn Version: 1.15.2
  • Node Version: 11.14.0
bug

Most helpful comment

We will lock it to 7.1.5 in kepler.gl.

All 5 comments

For those looking for a fix in the meantime, you can add the following to your package.json:

"resolutions": {
    "deck.gl": "7.1.5"
 }

and then re-run yarn install (note that npm does not support resolutions, so you must be using yarn)

We will lock it to 7.1.5 in kepler.gl.

Awesome, thanks @heshan0131! I went ahead and opened a PR for this to save you some time:
https://github.com/keplergl/kepler.gl/pull/688

Thanks for the workaround @treyhakanson! @heshan0131 any idea when can we expect an official patch for this?

keplerl.[email protected] is released with this

Was this page helpful?
0 / 5 - 0 ratings

Related issues

MorganGellert picture MorganGellert  路  5Comments

BioBricoleur picture BioBricoleur  路  3Comments

itch96 picture itch96  路  6Comments

1995parham picture 1995parham  路  4Comments

coenvalk picture coenvalk  路  5Comments