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

MeshTexture does not render correctly on GPUparticles2D, CPUparticles2D or Polygon2D #102031

Open
mk56-spn opened this issue Jan 25, 2025 · 4 comments

Comments

@mk56-spn
Copy link

Tested versions

  • Reproducible in the 4.4 beta.
  • Reproducible in 4.1.1 stable.

Honestly i doubt it ever worked given its relatively niche so its feasible it was never brought up ( seems to be the case from an issue search)

System information

Godot v4.4.beta.mono (d004aed08) - Pop!_OS 22.04 LTS on X11 - X11 display driver, Multi-window, 1 monitor - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4060 Laptop GPU - 12th Gen Intel(R) Core(TM) i7-12650H (16 threads) (this is the branch from this PR : #102028 but the issues are present on other versions too

Issue description

The MeshTexture 2D doesn't display on most the aforementioned nodes. There may be good reasons for this internally but if so it shouldn't show up in the dropdowns imo, or at least throw some sort of error upon attempted assignment

Steps to reproduce

Create a one of the aforementioned nodes, add a MeshTexture to it with it's corresponding mesh and texture. You will be greeted by a blank canvas,

I highly recommend anyone testing this issue test it on #102028 since meshtexture has a plethora of other bugs previous to that that will harm evaluation of this one potentially

Minimal reproduction project (MRP)

MeshParticlesPolygonTest.zip

@beicause
Copy link
Contributor

beicause commented Jan 25, 2025

The 3d model in the MRP is not suitable for MeshTexture, as it does not have appropriate UV. The simplest way to test is using QuadMesh.

@mk56-spn

This comment has been minimized.

@beicause
Copy link
Contributor

Sorry, please ignore my previous comment. I just test the model in #81213, not this MRP.

@mk56-spn
Copy link
Author

mk56-spn commented Jan 26, 2025

The most recent changes to #102028 would close this issue / proposal since it is now rendered in a viewport and therefor is a real texture suitable for usage anywhere a normal texture would be a requisite ( barring special cases like point lights that get atlased) , as appears to be the case here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants