image has numerous users when only 1 material actually is using it #42523

New Issue

Jacob Morris · 2014-11-06T14:59:39+01:00

Jacob Morris commented

2014-11-06 14:59:39 +01:00

Window 8.1

Blender Version
Broken: 2.72b 9e963ae

I have an addon that adds siding and you can add materials to that siding with it. Some of the types of siding it adds a image texture to and if you generate the material multiple times and check how many users the image has it says a high number, when really only one material is using it. My code works by adding in an extra material and then using that material to replace the one on the object and then deleting the temporary one. So I don't know if it is just not updating that the material is deleted and not using that image anymore? Here is my code:
JARCH_siding.zip

Steps:

Run "JARCH_siding.py" from the text editor
Add Siding "View 3D > Add > Mesh > JARCH Vis > Add Siding
Click "Cycles Materials"
Add image texture
Hit "Generate" several times, note how memory usage jumps
Go to the node editor and note the number of users on the image.
The part of code that adds the materials starts at line 1727

Window 8.1 **Blender Version** Broken: 2.72b 9e963ae I have an addon that adds siding and you can add materials to that siding with it. Some of the types of siding it adds a image texture to and if you generate the material multiple times and check how many users the image has it says a high number, when really only one material is using it. My code works by adding in an extra material and then using that material to replace the one on the object and then deleting the temporary one. So I don't know if it is just not updating that the material is deleted and not using that image anymore? Here is my code: [JARCH_siding.zip](https://archive.blender.org/developer/F122256/JARCH_siding.zip) Steps: 1. Run "JARCH_siding.py" from the text editor 2. Add Siding "View 3D > Add > Mesh > JARCH Vis > Add Siding 3. Click "Cycles Materials" 4. Add image texture 5. Hit "Generate" several times, note how memory usage jumps 6. Go to the node editor and note the number of users on the image. The part of code that adds the materials starts at line 1727

Jacob Morris commented

2014-11-06 14:59:39 +01:00

Changed status to: 'Open'

Jacob Morris commented

2014-11-06 14:59:39 +01:00

Added subscriber: @BlendingJake

Antonis Ryakiotakis commented

2014-11-06 15:38:13 +01:00

Changed status from 'Open' to: 'Archived'

Antonis Ryakiotakis closed this issue

2014-11-06 15:38:13 +01:00

Antonis Ryakiotakis self-assigned this 2014-11-06 15:38:13 +01:00

Antonis Ryakiotakis commented

2014-11-06 15:38:13 +01:00

Materials never get deleted from the database when you create them unless you close blender - if you search for them in the material search field you will find them. To make sure your image does not have any users you have to basically remove all references of the image from the temporary material. But again creating so many materials looks like a bad idea, you might want to consider caching them.

This looks like working as intended and patch is more than 2000 lines and -sorry to say- written like a wall of text so it's quite hard to figure out what is happening. It would be good to have a few steps with a blend file where image leaking can be demonstrated.

Materials never get deleted from the database when you create them unless you close blender - if you search for them in the material search field you will find them. To make sure your image does not have any users you have to basically remove all references of the image from the temporary material. But again creating so many materials looks like a bad idea, you might want to consider caching them. This looks like working as intended and patch is more than 2000 lines and -sorry to say- written like a wall of text so it's quite hard to figure out what is happening. It would be good to have a few steps with a blend file where image leaking can be demonstrated.

Antonis Ryakiotakis commented

2014-11-06 15:38:41 +01:00

Closing until a real leak can be shown.

Jacob Morris commented

2014-11-06 15:41:55 +01:00

@Psy-Fi
The thing is though that the material is being deleted. If you check bpy.data.materials the material is not there. Did you just run the code and see the issue? I understand that it is hard to read threw other peoples code and it is hard to simplify this down, as when you restart Blender the issue is gone, but to my understanding it shouldn't be holding onto it since the material is no longer listed.

@Psy-Fi The thing is though that the material is being deleted. If you check bpy.data.materials the material is not there. Did you just run the code and see the issue? I understand that it is hard to read threw other peoples code and it is hard to simplify this down, as when you restart Blender the issue is gone, but to my understanding it shouldn't be holding onto it since the material is no longer listed.

Antonis Ryakiotakis commented

2014-11-06 15:51:50 +01:00

Changed status from 'Archived' to: 'Open'

Antonis Ryakiotakis reopened this issue

2014-11-06 15:51:50 +01:00

Antonis Ryakiotakis commented

2014-11-06 15:51:50 +01:00

Reopening, confirmed. Materials seem to be deleted also

Jacob Morris commented

2014-11-06 15:53:09 +01:00

Added subscriber: @Psy-Fi

Jacob Morris commented

2014-11-06 15:53:09 +01:00

Thanks @Psy-Fi, I do hope to update my code, but this did seem to be a bug.

Antonis Ryakiotakis commented

2014-11-06 16:14:26 +01:00

Added subscriber: @ideasman42

Antonis Ryakiotakis commented

2014-11-06 16:14:26 +01:00

@ideasman42 I see this is caused by 916a58f4e

I don't have valgrind here to check for details but enabling with address sanitizer seems fine. Also if this is problematic on quit might be worth NULLing out the pointers first on exit or so?

@ideasman42 I see this is caused by 916a58f4e I don't have valgrind here to check for details but enabling with address sanitizer seems fine. Also if this is problematic on quit might be worth NULLing out the pointers first on exit or so?

Sergey Sharybin commented

2014-12-26 10:25:19 +01:00

Added subscriber: @Sergey

Sergey Sharybin commented

2014-12-26 10:25:19 +01:00

Well, user counters are only guaranteed to be correct after loading the file. Deleting datablocks might cause dangling users for until next save/reload. Making it so user counter is always correct is rather a huge topic and wouldn't really consider this is a bug.

Unless there are more issues here @Psy-Fi?

Well, user counters are only guaranteed to be correct after loading the file. Deleting datablocks might cause dangling users for until next save/reload. Making it so user counter is always correct is rather a huge topic and wouldn't really consider this is a bug. Unless there are more issues here @Psy-Fi?

Antonis Ryakiotakis commented

2014-12-26 11:50:33 +01:00

Changed status from 'Open' to: 'Archived'

Antonis Ryakiotakis closed this issue

2014-12-26 11:50:33 +01:00

Antonis Ryakiotakis commented

2014-12-26 11:50:33 +01:00

No, that should be it.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

image has numerous users when only 1 material actually is using it #42523