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
Audit module dependencies #8573
Conversation
7a39556
to
8cbd35e
Compare
@@ -40,6 +40,7 @@ | |||
"@loaders.gl/core": "^4.1.0", | |||
"@loaders.gl/csv": "^4.1.0", | |||
"@luma.gl/core": "9.0.0-beta.4", | |||
"d3-dsv": "^1.0.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.
@loaders.gl/csv
is missing this dependency @ibgreen
7120232
to
6347bad
Compare
1448a5d
to
6927e36
Compare
"@luma.gl/shadertools": "9.0.0-beta.4", | ||
"@math.gl/web-mercator": "^4.0.0", | ||
"d3-hexbin": "^0.2.1" | ||
}, | ||
"peerDependencies": { | ||
"@deck.gl/core": "^9.0.0-alpha", | ||
"@deck.gl/layers": "^9.0.0-alpha", | ||
"@luma.gl/core": "9.0.0-beta.4", | ||
"@luma.gl/engine": "9.0.0-beta.4" | ||
"@luma.gl/core": "^9.0.0-beta", |
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.
Users don't have to explicitly specify luma deps right? they are covered by deck.gl/core or something? Why is shadertools handled differently?
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.
Technically the only package that has to go into peer dep is core, which does not allow multiple copies.
@@ -509,20 +520,20 @@ export default class DataColumn<Options, State> { | |||
// https://developer.mozilla.org/en-US/docs/Web/API/WebGLRenderingContext/vertexAttribPointer | |||
normalizeConstant(value: NumericArray): NumericArray { |
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.
Should this always return a Float32Array
?
If we are using luma.gl types, should this be a luma.gl utility?
6347bad
to
7bbdd24
Compare
For #7457
Change List