Skip to content
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

fix #1760 - the phenomenon that data is not properly initialize #1765

Conversation

Seungup
Copy link
Collaborator

@Seungup Seungup commented Jul 13, 2023

Modify to call "setImageData3D" method even if "2d-array" in addition to "3d"

For #1760

Background

  • If the value of 'dimension' is '2d-array', the phenomenon that was incorrectly initialized by performing a basic initialization without initializing using the 'setImageData3D' method

Change List

  • If the value of 'dimension' is '3d' or '2d-array', call the 'setImageData3D' method and modify it to perform initialization.

Modify to call "setImageData3D" method even if "2d-array" in addition to "3d"
@Seungup Seungup added the bug label Jul 13, 2023
@Seungup Seungup self-assigned this Jul 13, 2023
@Seungup Seungup requested a review from ibgreen July 13, 2023 15:34
@Seungup
Copy link
Collaborator Author

Seungup commented Jul 13, 2023

The immediate phenomenon has been resolved, but the 'setImageData3D' method has not yet been added to the test case.

@ibgreen
Copy link
Collaborator

ibgreen commented Jul 13, 2023

Ideally we should add test cases, but happy to land this for now.

@ibgreen ibgreen merged commit c91e8f3 into master Jul 13, 2023
1 check passed
@ibgreen ibgreen deleted the 1760-if-dimension-is-2d-array-webgltexture-will-not-be-initialized-correctly branch July 13, 2023 16:09
@Seungup Seungup restored the 1760-if-dimension-is-2d-array-webgltexture-will-not-be-initialized-correctly branch July 15, 2023 02:37
@Seungup Seungup deleted the 1760-if-dimension-is-2d-array-webgltexture-will-not-be-initialized-correctly branch July 15, 2023 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

If "dimension" is "2d-array", 'WEBGLTexture' will not be initialized correctly.
2 participants