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

Print errors that make more sense #9488

Closed
CsloudX opened this issue Apr 10, 2024 · 9 comments
Closed

Print errors that make more sense #9488

CsloudX opened this issue Apr 10, 2024 · 9 comments

Comments

@CsloudX
Copy link

CsloudX commented Apr 10, 2024

Describe the project you are working on

NoGame APP

Describe the problem or limitation you are having in your project

Everty time I start my project, it will report error like this:
image
It like has 2 error:

  1. Some resource need load 'res://.godot/imported/atlastexture.dds-68d580e348c8dc76609729290d138ab5.ctex'.
  2. There was error image with 0 pixels.

I'm try solve this problems, I deleted .godot folder, but it not work.

Describe the feature / enhancement and how it helps to overcome the problem or limitation

Look the error log. I can't found which file want load 'res://.godot/imported/atlastexture.dds-68d580e348c8dc76609729290d138ab5.ctex' and I'm also can't found which image file with 0 pixels.
so i always has this error log here when start project.

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

Make error more sense, let me know which file want load 'res://.godot/imported/atlastexture.dds-68d580e348c8dc76609729290d138ab5.ctex' and which image file with 0 pixels so i can check this files.

If this enhancement will not be used often, can it be worked around with a few lines of script?

No

Is there a reason why this should be core and not an add-on in the asset library?

It should be core

@AThousandShips
Copy link
Member

AThousandShips commented Apr 10, 2024

Imported resources create their internal versions, so in this case it's referenced via atlastexture.dds

@CsloudX
Copy link
Author

CsloudX commented Apr 10, 2024

Imported resources create their internal versions, so in this case it's referenced via res://.godot/imported/atlastexture.dds

I'm look the res://.godot/imported folder. where was no atlastexture.dds and res://.godot/imported/atlastexture.dds-68d580e348c8dc76609729290d138ab5.ctex file.
image

@AThousandShips
Copy link
Member

AThousandShips commented Apr 10, 2024

The atlastexture.dds isn't there, it's in some other folder, my bad mistyped, it should be elsewhere :)

It might not be known to the editor where it's used though, as the editor might link directly to it, unsure without more details

@CsloudX
Copy link
Author

CsloudX commented Apr 10, 2024

The atlastexture.dds isn't there, it's in some other folder, my bad mistyped, it should be elsewhere :)

It might not be known to the editor where it's used though, as the editor might link directly to it, unsure without more details

Thanks, YES. I'm found the atlastexture.dds in my project folder.

@CsloudX
Copy link
Author

CsloudX commented Apr 10, 2024

** ADDTION: **
Thanks @AThousandShips , I can search atlastexture.dds in my project.
So, only the point 2 need solve.

@AThousandShips
Copy link
Member

That is likely not possible to clearly error, as Image isn't guaranteed to know its size, but without more details on what you're doing I can't tell

@lufog
Copy link

lufog commented Apr 11, 2024

While poking around other people's demos and examples, I myself often encountered warnings/errors like “invalid UID, the path is being used instead” or something wrong with the animation track/mesh/material/etc. (I’m writing from memory). And since in such errors in almost all cases it is not written which file specifically causes it, only the path to the cpp file of the engine. You have to open scenes/scripts at random, trying to find the reason.

I realize that my vague description is not very helpful. But it’s difficult to immediately provide a complete list of such non-obvious warnings/errors. In the future, as I encounter them again, I will try to collect them somewhere with a minimal description and post them all at once, either here or in a new proposal.

@fire fire changed the title Make error more sense Print errors that make more sense Apr 11, 2024
@CsloudX
Copy link
Author

CsloudX commented Jun 14, 2024

For ** There was error image with 0 pixels.**
I'm try debug with source. and found it because i'm create a error altas.
image
Note the p_metadata, it was error data.
IMO, we can add a error message check the generated. if it was null, prints the error and output the p_item.path, then, i can know which file was error.

ADDTION: godotengine/godot#93151

@AThousandShips
Copy link
Member

Consolidating in:

@AThousandShips AThousandShips closed this as not planned Won't fix, can't repro, duplicate, stale Jul 13, 2024
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