-
Notifications
You must be signed in to change notification settings - Fork 62
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
refactor manifests to be lazier #173
base: 09-30-uv_for_quadfeather
Are you sure you want to change the base?
refactor manifests to be lazier #173
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
aac2391
to
0491be5
Compare
@@ -373,6 +373,7 @@ export class Scatterplot { | |||
tileProxy, | |||
}: DS.DataSpec): Promise<DS.Deeptable> { | |||
if (source_url !== undefined) { | |||
console.log('Loading from source url', source_url); |
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 unnecessary console.log statements to avoid cluttering the console and potentially leaking sensitive information.
console.log('Loading from source url', source_url); |
0491be5
to
cb409c1
Compare
Testing scaling to 10 billion points.
Important
Refactor tile manifest handling in Deepscatter to use a flat structure, improving scalability and performance, with updates to rendering and testing components.
Deeptable
andTile
to use a flat manifest structure for improved scalability.fromQuadfeather()
inDeeptable.ts
to load flat manifests.points()
method fromDeeptable.ts
as part of refactor.ReglRenderer
inregl_rendering.ts
to accommodate changes in tile manifest structure.render_points()
to usetile.metadata.nPoints
instead oftile.manifest.nPoints
.tix.spec.ts
to test tile index conversion functions.package.json
to includevitest
for testing.This description was created by for cb409c1. It will automatically update as commits are pushed.