Can I use part of a sprite as the light texture?

[crayonape]

Tested versions

4.3rc2

System information

MacOS Monterey 12.7.5

Issue description

Sorry, I know this isn't the right place, but I've asked around and no one has given me a definitive answer.

When I try to add light textures to my light nodes, I find that light textures are a bit different from Sprite2D textures that can be cut from large sprites.
Currently, I create separate files for each light texture, but this is a bit inconvenient because I have too many light textures and if I create a file for each texture, there will be a lot of files.
My question is, am I doing something wrong or is it really only possible to generate separate texture files for each light node?

Steps to reproduce

N/A

Minimal reproduction project (MRP)

N/A

[Summersay415]

What about AtlasTexture?

[crayonape]

What about AtlasTexture?

I tried this, but for some reason unknown to me the lights don't show up when I use an AtlasTexture.

[AThousandShips]

This is a limitation that will be enforced, see:

• Only allow valid types in Decal, Light3D projector, PointLight2D texture and CSGMesh3D mesh #88349

Keeping this open to track this specific case and so it can be documented

[crayonape]

This is a limitation that will be enforced, see:

• Only allow valid types in Decal, Light3D projector and PointLight2D texture #88349

Keeping this open to track this specific case and so it can be documented

So currently I can't clip part of a sprite to a Light2D texture, right?

[huwpascoe]

So currently I can't clip part of a sprite to a Light2D texture, right?

Correct. It probably won't change in future either.

A workaround could be to create an adapter resource like

@tool
class MyAdapter
extends Texture2D

static var _cache: Dictionary # store textures using weakref()

@export var source: AtlasTexture
@export var some_way_to_select_subimage

var _tex: Texture2D

func _draw(...):
  if not _tex:
   _tex = _create_or_load_from_cache(
      source, some_way_to_select_subimage)
  _tex.draw(...)

# and the other required Texture2D virtual methods
...

[crayonape]

So currently I can't clip part of a sprite to a Light2D texture, right?

Correct. It probably won't change in future either.

A workaround could be to create an adapter resource like

@tool
class MyAdapter
extends Texture2D

static var _cache: Dictionary # store textures using weakref()

@export var source: AtlasTexture
@export var some_way_to_select_subimage

var _tex: Texture2D

func _draw(...):
  if not _tex:
   _tex = _create_or_load_from_cache(
      source, some_way_to_select_subimage)
  _tex.draw(...)

# and the other required Texture2D virtual methods
...

Then I think my project is ready for like a thousand png files.

[Calinou]

Then I think my project is ready for like a thousand png files.

In terms of memory usage, it'll be the same as using a single large PNG file anyway. File size on disk will also nearly be the same - the only difference is that lossless compression will be a bit less efficient, and there will be more PNG headers. Either way, I expect the difference to be pretty small (less than a MB compared to a single large atlas).

[crayonape]

In terms of memory usage, it'll be the same as using a single large PNG file anyway. File size on disk will also nearly be the same - the only difference is that lossless compression will be a bit less efficient, and there will be more PNG headers. Either way, I expect the difference to be pretty small (less than a MB compared to a single large atlas).

I'm not really worried about the file size, it's just that having too many files makes it hard to organize the project. Too many small files sometimes make it hard to find the files for my light node.

[crayonape]

I really want light textures to use partial sprites. Take this stall for example, the lantern can illuminate the player, but the shop sign can't, they have different item_cull_mask, so I have to create two files. If I want to add more complex lighting to this stall, maybe I need more light files to achieve a small stall. And my game will have many such stalls, so it is foreseeable that the file system will cause a mess. I'm really curious why light texture can't use partial sprites like Sprite2D, is there any technical difficulty?

[Calinou]

Discussion opened: godotengine/godot-proposals#10487

I'm really curious why light texture can't use partial sprites like Sprite2D, is there any technical difficulty?

Yes, atlas support needs dedicated support in each node. This can add a lot of maintenace burden, so it was only implemented in places where it's really needed (e.g. to save on memory).