Video Texture (bge.texture) ImageRender / ImageMirror broken #30173

New Issue

Thomas Szepe · 2012-02-13T22:41:16+01:00

Thomas Szepe commented

2012-02-13 22:41:16 +01:00

%%%On all Blender versions (I think) higher than 2.61.4 r43747, the ImageRender and ImageMirror from the video texture module did not work any more.

Last test on Blender 2.61.4 r44098, ImageRender and ImageMirror still not working.

Windows XP 32 Bit.
%%%

%%%On all Blender versions (I think) higher than 2.61.4 r43747, the ImageRender and ImageMirror from the video texture module did not work any more. Last test on Blender 2.61.4 r44098, ImageRender and ImageMirror still not working. Windows XP 32 Bit. %%%

Thomas Szepe commented

2012-02-13 22:41:16 +01:00

Changed status to: 'Open'

Dalai Felinto commented

2012-02-13 22:47:06 +01:00

%%%strange, I recall Benoit fixing ImageRender.
ImageMirror I'm using in a project and seems to be working fine.

confirmed that this file render black - OSX svn > 2.62rc%%%

%%%strange, I recall Benoit fixing ImageRender. ImageMirror I'm using in a project and seems to be working fine. confirmed that this file render black - OSX svn > 2.62rc%%%

Thomas Szepe commented

2012-02-14 18:51:42 +01:00

%%%I have uploaded a the mirror example which I used for testing.%%%

Dalai Felinto commented

2012-02-14 19:17:25 +01:00

%%%interesting, the bug is actually that video texture seems to be buggy at frame 1.
change your sensors to a delay sensor with delay one and they both work%%%

%%%interesting, the bug is actually that video texture seems to be buggy at frame 1. change your sensors to a delay sensor with delay one and they both work%%%

Thomas Szepe commented

2012-02-14 19:45:00 +01:00

%%%Confirmed .
With the delay sensor it is working.%%%

%%%Confirmed . With the delay sensor it is working.%%%

Benoit Bolsee commented

2012-02-18 16:27:01 +01:00

%%%This bug is caused by the canvas display area being initialized on render frame 1, which happens after logic frame 1.
Fixed by initializing the area at canvas creation.

Fixed in revision 44225
%%%

%%%This bug is caused by the canvas display area being initialized on render frame 1, which happens after logic frame 1. Fixed by initializing the area at canvas creation. Fixed in revision 44225 %%%

Thomas Szepe commented

2012-02-19 18:07:23 +01:00

%%%Thank you for fixing this bug.
But I think you have forgotten to fix the blenderplayer.%%%

%%%Thank you for fixing this bug. But I think you have forgotten to fix the blenderplayer.%%%

Benoit Bolsee commented

2012-02-20 10:18:27 +01:00

%%%In fact I fixed the blenderplayer, but as I'm testing it, I realize that the logic doesn't work at all: I seems that the logic bricks are not loaded with the file. I will check that asap%%%

Mitchell Stokes commented

2012-11-27 19:34:36 +01:00

%%%I'm going to reopen this until the player is fixed.%%%

Benoit Bolsee commented

2012-12-05 21:43:10 +01:00

%%%As it seems the player now works correctly: the blend file works well in the player. Can you verify that before I close the ticket?%%%

Thomas Szepe commented

2012-12-06 22:07:50 +01:00

%%%I am sorry. The both files that I uploaded before was only to show the first bug that you have fixed. With this files you can't see the Blenderplayer bug.
I uploaded a new file .

Working with Blender

Working with Blenderplayer if a always sensor has pulse mode is on.
Not working with Blenderplayer if a always sensor has pulse mode is off.
Working with Blenderplayer if a delaysensor has a delay of 1 and pulse mode is off.

Not working Blenderplayer if the player is started with the start button in the standalone player panel.
It seams to be a path problem. I made all paths relative, but If I start the player with these button the paths will change back to absolute on Blender 2.64a.
Blender 2.94.9 r52796 don't change the texture phat but in the console I get the same error.
Path cant be made relative "D:.....\Blender Files\RealtimeTextureSwitching\Happy.png"
%%%

%%%I am sorry. The both files that I uploaded before was only to show the first bug that you have fixed. With this files you can't see the Blenderplayer bug. I uploaded a new file . Working with Blender Working with Blenderplayer if a always sensor has pulse mode is on. Not working with Blenderplayer if a always sensor has pulse mode is off. Working with Blenderplayer if a delaysensor has a delay of 1 and pulse mode is off. Not working Blenderplayer if the player is started with the start button in the standalone player panel. It seams to be a path problem. I made all paths relative, but If I start the player with these button the paths will change back to absolute on Blender 2.64a. Blender 2.94.9 r52796 don't change the texture phat but in the console I get the same error. Path cant be made relative "D:\.....\Blender Files\RealtimeTextureSwitching\Happy.png" %%%

Benoit Bolsee commented

2012-12-09 01:06:19 +01:00

%%%The problem in the player is different than the original ImageRender problem: in the player, the image textures are only initialized in the GPU on the first render frame, which happens after the first logic frame, so that the texture created by VideoTexture in the first frame are deleted and overwritten by the GPU.

To work around this problem, VideoTexture can force a GPU load before swapping the texture id. This works for simple image texture, not necessarily with more complex images (sequence, movies,..) because they can have multiple buffers and VideoTexture has no way to tell which buffer the GPU will need. I don't think this is an issue though: these complex images will in any case interfere badly with VideoTexture, they should not be used as base for swapping a new Texture.

Note that the problem does not exist when using materials (texture.materialID(obj, 'MA...')) because the game engine initializes the materials during conversion.

I did not commit the patch yet because Blender is RC phase. The patch is available here: http://www.pasteall.org/37856/diff
%%%

%%%The problem in the player is different than the original ImageRender problem: in the player, the image textures are only initialized in the GPU on the first render frame, which happens after the first logic frame, so that the texture created by VideoTexture in the first frame are deleted and overwritten by the GPU. To work around this problem, VideoTexture can force a GPU load before swapping the texture id. This works for simple image texture, not necessarily with more complex images (sequence, movies,..) because they can have multiple buffers and VideoTexture has no way to tell which buffer the GPU will need. I don't think this is an issue though: these complex images will in any case interfere badly with VideoTexture, they should not be used as base for swapping a new Texture. Note that the problem does not exist when using materials (texture.materialID(obj, 'MA...')) because the game engine initializes the materials during conversion. I did not commit the patch yet because Blender is RC phase. The patch is available here: http://www.pasteall.org/37856/diff %%%

Thomas Szepe commented

2012-12-11 21:57:54 +01:00

%%%With your patch the player is now working with my test file.

But there are two new memory leaks now .
ImBuf_struct len: 2304 02AD3418
imb_addrectImBuf len: 16384 06810020%%%

%%%With your patch the player is now working with my test file. But there are two new memory leaks now . ImBuf_struct len: 2304 02AD3418 imb_addrectImBuf len: 16384 06810020%%%

Benoit Bolsee commented

2012-12-12 23:51:01 +01:00

%%%The memory leak is actually not the fault of VideoTexture: if you manually close (delete) the texture before quitting the game you will see that there is no leak.

The fact is that the player does not automatically clean the logic object when leaving the game. This is a different problem that should be reported separately.%%%

%%%The memory leak is actually not the fault of VideoTexture: if you manually close (delete) the texture before quitting the game you will see that there is no leak. The fact is that the player does not automatically clean the logic object when leaving the game. This is a different problem that should be reported separately.%%%

Thomas Szepe commented

2012-12-16 20:25:14 +01:00

%%%Yes I know, but the funny thing is that without the patch the memory will automatically cleaned.

I have made a bug report [#32708] two month ago and I think that is the same issue.%%%

%%%Yes I know, but the funny thing is that without the patch the memory will automatically cleaned. I have made a bug report [#32708] two month ago and I think that is the same issue.%%%

Dalai Felinto commented

2013-04-08 04:19:35 +02:00

%%%Benoit, can your patch be committed?%%%