Skip to content

Conversation

@ryantrem
Copy link
Member

Merging #16730 into #16709 and testing the build.

Georgina and others added 30 commits June 3, 2025 11:45
…dd controlled state to syncedSlider; remove index from dropdown key; ToolHostProps use PropsWithChildren; restore package.json;
… wrapper, enable in NME, test if build fails
@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

Reviewer - this PR has made changes to one or more package.json files.

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

Building or testing the sandbox has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16747/merge/testResults/

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

You have made possible changes to the playground.
You can test the snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16747/merge/

The snapshot playground with the CDN snapshot (only when available):

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16747/merge/?snapshot=refs/pull/16747/merge

Note that neither Babylon scenes nor textures are uploaded to the snapshot directory, so some playgrounds won't work correctly.

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

You have changed file(s) that made possible changes to the sandbox.
You can test the sandbox snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16747/merge/

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 12, 2025

Reviewer - this PR has made changes to one or more package.json files.

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 13, 2025

You have made possible changes to the playground.
You can test the snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16747/merge/

The snapshot playground with the CDN snapshot (only when available):

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16747/merge/?snapshot=refs/pull/16747/merge

Note that neither Babylon scenes nor textures are uploaded to the snapshot directory, so some playgrounds won't work correctly.

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 13, 2025

You have changed file(s) that made possible changes to the sandbox.
You can test the sandbox snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16747/merge/

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 13, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Jun 13, 2025

deltakosh pushed a commit that referenced this pull request Jun 16, 2025
….json to point to minified bundle (#16759)

When introducing fluent dependency in [this
PR](#16709) we saw our build
agents running out of memory when running es6/UMD build steps, and we
also saw the importing of public inspector package causing heap
allocation error in the test app sceneWithInspector test.

After many attempts to solve (branching off onto another [smaller
PR](#16747) to isolate the
causes) the solution we are landing on is to reduce parallelization of
build steps (solving OOM during es6/UMD build, and confirmed that it
wont slow down our build) and to update the inspector and gui editor
package.json to point to the minified bundle (solving heap allocation
error, and confirmed via npm link + babylonjs-webpack-es6 test app that
inspector loads properly)

Co-authored-by: Georgina <[email protected]>
@deltakosh
Copy link
Contributor

Can we close that one @ryantrem ?

@ryantrem ryantrem closed this Jun 17, 2025
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.

5 participants