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
[Chore]: Technical: Isolate deckgl-layers #1851
Conversation
8eca205
to
4b385de
Compare
src/utils/export-utils.ts
Outdated
@@ -36,6 +36,7 @@ import {set, generateHashId} from 'utils/utils'; | |||
|
|||
import {createIndexedDataContainer} from './table-utils/data-container-utils'; | |||
import {ExportImage} from 'reducers/ui-state-updaters'; | |||
import { VisState } from 'reducers'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove space here
src/deckgl-layers/babel.config.js
Outdated
const PLUGINS = [ | ||
['@babel/plugin-transform-typescript', {isTSX: true, allowDeclareFields: true}], | ||
'@babel/plugin-transform-modules-commonjs', | ||
'@babel/plugin-proposal-export-namespace-from', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you need this plugin?
src/deckgl-layers/package.json
Outdated
], | ||
"dependencies": { | ||
"colorbrewer": "^1.5.0", | ||
"d3-scale": "^3.2.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would think you need lot os deck.gl dependencies from @deck.gl
src/deckgl-layers/package.json
Outdated
"@mapbox/geo-viewport": "^0.4.1", | ||
"@mapbox/vector-tile": "^1.3.1", | ||
"@types/geojson": "^7946.0.7", | ||
"colorbrewer": "^1.5.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is colorbrewer
needed?
src/deckgl-layers/package.json
Outdated
"@types/geojson": "^7946.0.7", | ||
"colorbrewer": "^1.5.0", | ||
"d3-array": "^2.8.0", | ||
"d3-scale": "^3.2.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is d3-scale
needed?
src/deckgl-layers/package.json
Outdated
"d3-scale": "^3.2.3", | ||
"global": "^4.3.0", | ||
"keymirror": "^0.1.1", | ||
"react-map-gl-draw": "0.14.8", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not needed
src/deckgl-layers/src/index.ts
Outdated
|
||
export {default as EnhancedHexagonLayer} from './hexagon-layer/enhanced-hexagon-layer'; | ||
|
||
export * from './layer-utils/cluster-utils'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need to export this
src/deckgl-layers/src/index.ts
Outdated
@@ -0,0 +1,20 @@ | |||
export {default as ThreeDBuildingLayer} from './3d-building-layer/3d-building-layer'; | |||
export * from './3d-building-layer/3d-building-utils'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't need to export this
src/deckgl-layers/src/index.ts
Outdated
@@ -0,0 +1,20 @@ | |||
export {default as ThreeDBuildingLayer} from './3d-building-layer/3d-building-layer'; | |||
export * from './3d-building-layer/3d-building-utils'; | |||
export * from './3d-building-layer/types'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move type export to the bottom
src/deckgl-layers/src/index.ts
Outdated
export {default as EnhancedHexagonLayer} from './hexagon-layer/enhanced-hexagon-layer'; | ||
|
||
export * from './layer-utils/cluster-utils'; | ||
export * from './layer-utils/cpu-aggregator'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't need to export this
src/deckgl-layers/src/index.ts
Outdated
export * from './layer-utils/cpu-aggregator'; | ||
export * from './layer-utils/shader-utils'; | ||
|
||
export {default as EnhancedLineLayer} from './line-layer/line-layer'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move all layer exports to the top
src/deckgl-layers/src/index.ts
Outdated
|
||
export {default as EnhancedLineLayer} from './line-layer/line-layer'; | ||
|
||
export * from './svg-icon-layer/scatterplot-icon-layer'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't need to export this
src/deckgl-layers/src/index.ts
Outdated
export {default as EnhancedLineLayer} from './line-layer/line-layer'; | ||
|
||
export * from './svg-icon-layer/scatterplot-icon-layer'; | ||
export {default as SvgIconLayer} from './svg-icon-layer/svg-icon-layer'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
move all layer export to the top
fb9d5e9
to
0caaa6c
Compare
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
Signed-off-by: Daria Terekhova <daria.terekhova@actionengine.com>
0caaa6c
to
1fc175f
Compare
No description provided.