Skip to content

Conversation

@sunag
Copy link
Collaborator

@sunag sunag commented Apr 5, 2023

Description

Variables lodFudge and fullSize unused.

@github-actions
Copy link

github-actions bot commented Apr 5, 2023

📦 Bundle size

Full ESM build, minified and gzipped.

Filesize Gzipped Diff from dev
626.8 kB 155.8 kB -147 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Filesize Gzipped Diff from dev
418.5 kB 101.9 kB -147 B

@mrdoob
Copy link
Owner

mrdoob commented Apr 6, 2023

I don't think you meant to remove fullSize?

@mrdoob
Copy link
Owner

mrdoob commented Apr 6, 2023

I'm surprised the E2E tests passed...

@LeviPesin
Copy link
Contributor

I think @sunag removed fullSize from the both calls where it was used, so everything's OK.

@mrdoob
Copy link
Owner

mrdoob commented Apr 6, 2023

Oh, I see. The PR description confused me. I'll fix it.

@mrdoob mrdoob added this to the r152 milestone Apr 6, 2023
@mrdoob mrdoob merged commit 185df04 into mrdoob:dev Apr 6, 2023
@sunag sunag deleted the dev-webgl-cleanup branch April 6, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants