TSL: Different params order if method chaining is used. #25683
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merged
sunag
added a commit
to sunag/three.js
that referenced
this pull request
Mar 20, 2023
* CodeNode: Added language and serialize. * TSL: Renamed inversesqrt -> inverseSqrt, faceforward -> faceForward, * TSL: Added saturate * TSL: Different params order if method chaining is used. * Nodes: Change mix order and added FogNode.mixAssign()
sunag
added a commit
that referenced
this pull request
Mar 20, 2023
* CodeNode: Added language and serialize. * TSL: Renamed inversesqrt -> inverseSqrt, faceforward -> faceForward, * TSL: Added saturate * TSL: Different params order if method chaining is used. * Nodes: Change mix order and added FogNode.mixAssign() * Node: Ignore private properties on serialization. * Nodes: Added string, arrayBuffer * Node: Added .getSerializeChildren() * Added ScriptableNode * Added scriptable example and serialization test. * update style * cleanup * cleanup * Code Based Node moved to /code/ * ScriptableNode: Fix async refresh, added .setLocal(), .getLocal() * added async scriptable and serialization test * optional output * added output example * ScriptableNode: Move editor property to function call if needed. * ScriptableValueNode: Added support to more CPU nodes. * ShaderNode: Reduces the use of unnecessary conversions. * webgpu_materials: Added CPU parameter as node. * ScriptableNode: Move THREE, TSL to global environment vars. * webgpu_materials: set globals * cleanup * cleanup * Revert "cleanup" This reverts commit 0cf0695. * Revert "fix conflicting" This reverts commit baaa14b, reversing changes made to 0923c2e. * PointerLockControls: Refactor class structure. (#25665) * Bump eslint from 8.35.0 to 8.36.0 (#25666) Bumps [eslint](https://github.com/eslint/eslint) from 8.35.0 to 8.36.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md) - [Commits](eslint/eslint@v8.35.0...v8.36.0) --- updated-dependencies: - dependency-name: eslint dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Update webgl_loader_md2.html (#25670) Double index increment preventing all animations to be set in UI. * Fix the Chinese translation of MeshToonMaterial (#25672) In the original version, the Chinese translation of MeshToonMaterial documents did not work properly. This update aims to fix this error * Bump puppeteer-core from 19.7.4 to 19.7.5 (#25671) Bumps [puppeteer-core](https://github.com/puppeteer/puppeteer) from 19.7.4 to 19.7.5. - [Release notes](https://github.com/puppeteer/puppeteer/releases) - [Changelog](https://github.com/puppeteer/puppeteer/blob/main/release-please-config.json) - [Commits](puppeteer/puppeteer@puppeteer-core-v19.7.4...puppeteer-core-v19.7.5) --- updated-dependencies: - dependency-name: puppeteer-core dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Create renovate.json (#25636) * Create renovate.json * Update renovate.json --------- Co-authored-by: mrdoob <[email protected]> * Delete dependabot.yml * Src: export AnimationAction (#25676) * Fix a document translation omission (#25677) * Create a translation document of DataArrayTexture Create a Chinese translation document of DataArrayTexture * Update DataArrayTexture.html --------- Co-authored-by: Michael Herzog <[email protected]> * Delete DataTexture2DArray.html * CSS2DRenderer: Introduce CSS2DObject.center (#25673) * CSS2DRenderer: Introduce CSS2DObject.center * Update css2d_label screenshot * fix gui * Vector2: Add `angleTo()`. (#25678) * GLTFLoader: Prevents set data uri in texture.name. (#25682) * GLTFLoader: Prevents set data uri in texture.name. * cleanup * fix maybe optional value * cleanup * Object3D: Honor `up` in `toJSON()` and `ObjectLoader`. (#25687) * WebGLRenderer: Add `onShaderError()`. (#25679) * Updated builds. * 3DMLoader: Clean up (#25423) * 3DMLoader: Clean up * Update 3DMLoader.js * Update 3DMLoader.js * Revert materials comparison change * Fix `elements` * Update 3DMLoader.js * Update 3DMLoader.js * Update 3DMLoader.js --------- Co-authored-by: Michael Herzog <[email protected]> * Examples: Fix webgl_geometry_spline_editor.html (#25690) Call render() after detach so that the tooltip disappears when clicking away from the selected helper point, as one would expect. * TSL: Different params order if method chaining is used. (#25683) * CodeNode: Added language and serialize. * TSL: Renamed inversesqrt -> inverseSqrt, faceforward -> faceForward, * TSL: Added saturate * TSL: Different params order if method chaining is used. * Nodes: Change mix order and added FogNode.mixAssign() --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Michael Herzog <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: llzen44 <[email protected]> Co-authored-by: 幽离 <[email protected]> Co-authored-by: Don McCurdy <[email protected]> Co-authored-by: mrdoob <[email protected]> Co-authored-by: 林炳权 <[email protected]> Co-authored-by: WestLangley <[email protected]> Co-authored-by: Levi Pesin <[email protected]> Co-authored-by: puqeko <[email protected]>
This was referenced Apr 25, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
I renamed
inverseSqrt,faceForwardand addedsaturateagain.@LeviPesin
inverseSqrt,faceForwardIt was a suggestion from you earlier and I think it makes sense.Different order of parameters can help in method chaining, and we also preserve the classic function signature if used in C-style.