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

ERROR: The Image width specified (0 pixels) must be greater than 0 pixels. #93151

Closed
CsloudX opened this issue Jun 14, 2024 · 5 comments · Fixed by #93255
Closed

ERROR: The Image width specified (0 pixels) must be greater than 0 pixels. #93151

CsloudX opened this issue Jun 14, 2024 · 5 comments · Fixed by #93255

Comments

@CsloudX
Copy link

CsloudX commented Jun 14, 2024

Tested versions

v4.3.dev6.mono.official [89850d5]

System information

Godot v4.3.dev6.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 5GB (NVIDIA; 31.0.15.3623) - AMD Ryzen 9 3900X 12-Core Processor (24 Threads)

Issue description

It report error when start project:
image
Note: it only report in the console, the Editor's output no error print.

Steps to reproduce

just start the MRP.

Minimal reproduction project (MRP)

ddstest.zip

Other steps:

  1. create a project.
  2. import a dds file.
  3. create altastexture.
  4. set atlastexture.texture with the dds
  5. restart project with console
@AThousandShips
Copy link
Member

Please try with beta1 to make sure it's still present

@CsloudX
Copy link
Author

CsloudX commented Jun 14, 2024

Please try with beta1 to make sure it's still present

Yes, it's still present.
image

Note: for big project(about 2000 files). the Editor's output window will report this error too.
godotengine/godot-proposals#9488

@akien-mga
Copy link
Member

akien-mga commented Jun 14, 2024

Confirmed in current master (680e489) with the MRP, as well as in 4.2-stable.

You can solve it by actually giving a non-zero sized region to the AtlasTexture:

image

Which is probably what you'd want in this scenario anyway.

The problem is a UX issue, that the AtlasTexture is created with an empty region which triggers an error.

It's not specific to using a DDS file, the same happens with using icon.svg as the Atlas texture.

@CsloudX
Copy link
Author

CsloudX commented Jun 14, 2024

IMO, 2 tasks need to do:

  1. it report error only in console, no in the output.
  2. the error not make sense which file has error, need enhence.

@AThousandShips
Copy link
Member

The error message part is already handled in your proposal so let's focus on the UX part here only

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

Successfully merging a pull request may close this issue.

3 participants