deck.gl icon indicating copy to clipboard operation
deck.gl copied to clipboard

[Bug] CARTO: columns property is missing in VectorQuerySourceOptions

Open padawannn opened this issue 4 months ago • 2 comments

Description

VectorQuerySourceOptions also must accept columns property

Flavors

  • [ ] Script tag
  • [ ] React
  • [ ] Python/Jupyter notebook
  • [ ] MapboxOverlay
  • [ ] GoogleMapsOverlay
  • [X] CartoLayer
  • [ ] ArcGIS

Expected Behavior

The Maps API must be called with columns parameter for vectorQuerySource dataset

Steps to Reproduce

Loads a vectorQuerySource setting the columns property

Environment

Logs

No response

padawannn avatar Feb 20 '24 15:02 padawannn

What about the h3QuerySource, boundaryQuerySource and quadbinQuerySource?

felixpalmer avatar Feb 20 '24 15:02 felixpalmer

What about the h3QuerySource, boundaryQuerySource and quadbinQuerySource?

We also need it for boundaryQuerySource but for h3QuerySource and, quadbinQuerySource makes no sense because the maps API needs to know how to aggregate columns and we have the aggregationExp for this

padawannn avatar Feb 20 '24 15:02 padawannn