Gimp plugin

hitchhikr · Post by **hitchhikr** » Tue Jun 13, 2006 12:15 pm

http://perso.orange.fr/franck.charlet/PSPTex.zip

PSPTex is a save plugin for GIMP 2.2.x allowing to create textures for the PlayStation Portable.

It can save raw RGBA (5650, 5551, 4444 or 8888) & indexed (T4, T8) textures (with 5650, 5551, 4444 or 8888 palette modes support) as well as DXTC (DXT1, DXT3, DXT5) compresssed ones, it supports the PSP textures swizzle mode and can generate mipmaps, it also have a color key feature to handle transparency and can convert grayscale pictures into alpha channel only textures.

(Note: if the plugin doesn't work under windows you'll need to copy the gimp dlls into the system directory).

white rabbit · Post by **white rabbit** » Tue Jun 13, 2006 10:53 pm

Thanks for that - I was thinking of making a texture editor, but was putting it off with all the work required.

I'll give that a go tonight!

hitchhikr · Post by **hitchhikr** » Wed Jun 14, 2006 2:26 am

I modified the description as this is not a texture editor but only a plugin allowing you to save pictures as raw datas in formats suitable for the psp.

Post by **Drakonite** » Wed Jun 14, 2006 10:24 am

Do you plan on releasing the source some time?

hitchhikr · Post by **hitchhikr** » Wed Jun 14, 2006 4:29 pm

Prolly in a couple of days, just the time to cleanup the source code.

white rabbit · Post by **white rabbit** » Wed Jun 14, 2006 7:10 pm

Sorry, that's pretty much what I meant - I can edit the texture data outside of the image without too much hassle, but editing an image in a knocked up app is never fun.

hitchhikr · Post by **hitchhikr** » Thu Jun 15, 2006 6:47 am

New version released including source code, same url as mentioned before.

Post by **Drakonite** » Sun Jun 18, 2006 6:03 pm

I've hacked it up a bit and have gotten it running on the linux version of gimp (developer version 2.3.8)

I've ran into a couple problems though...

The image is upside down (height-(y+1) used instead of y) I know this is often done in windows bmp files, but I don't think this is a good thing for the psp? ... was this intentional or is it a bug?

It seems it's supposed to automatically resize the image to a power of 2 width, however it does not appear to be working... I don't have time to look into why right now unfortunatly.

Also, do you have any intentions of adding support for saving the alpha channel?

hitchhikr · Post by **hitchhikr** » Sun Jun 18, 2006 6:45 pm

The image is upside down (height-(y+1) used instead of y) I know this is often done in windows bmp files, but I don't think this is a good thing for the psp? ... was this intentional or is it a bug?

I'm copying the texture from top to bottom, maybe your uv coordinates are upside down ? In the worst case, flip it vertically within gimp :D
If it's really a problem, i may add an option to allow reverting the textures for saving them, eventually.

It seems it's supposed to automatically resize the image to a power of 2 width, however it does not appear to be working... I don't have time to look into why right now unfortunatly.

No it's not, there's no reason why it should do that, the pow stuff is only used to generate the xxx_TEXTURE_WIDTH constant which is compliant with the textures sizes passed to the psp, the xxx_TEXTURE_REAL_WIDTH contains the real pixels width (as implied), both are to be passed to sceGuTexImage(). The plugin generates a .h file beside the .psptex as an helper.

Also, do you have any intentions of adding support for saving the alpha channel?

There's full support for alpha channel already (unless i screwed something up somewhere).

Post by **Drakonite** » Sun Jun 18, 2006 9:23 pm

hitchhikr wrote: I'm copying the texture from top to bottom, maybe your uv coordinates are upside down ? In the worst case, flip it vertically within gimp :D
If it's really a problem, i may add an option to allow reverting the textures for saving them, eventually.

So it's just my imagination that line 275 is the following? ;)

Code: Select all

offset = &#40;&#40;&#40;height - &#40;y + 1&#41;&#41; * width&#41; + x&#41;;

No it's not, there's no reason why it should do that, the pow stuff is only used to generate the xxx_TEXTURE_WIDTH constant which is compliant with the textures sizes passed to the psp, the xxx_TEXTURE_REAL_WIDTH contains the real pixels width (as implied), both are to be passed to sceGuTexImage(). The plugin generates a .h file beside the .psptex as an helper.

Erm.. don't you need the buffer width to be a factor of 2?
Even if you do, not neccisarily saying it's something it should do for you, just mentioning ;)

There's full support for alpha channel already (unless i screwed something up somewhere).

Doesn't seem to work for me... which could be anything from me screwying up to a change in gimp, to you screwying up ;) ... One way or another I'll look into it more after the compo.

hitchhikr · Post by **hitchhikr** » Sun Jun 18, 2006 9:53 pm

So it's just my imagination that line 275 is the following? ;)
Code:
Code: Select all
  offset = &#40;&#40;&#40;height - &#40;y + 1&#41;&#41; * width&#41; + x&#41;;

Damn you're right :D
I must have used this for a test purpose but i can't remember which one now, i'll modify that.

Erm.. don't you need the buffer width to be a factor of 2?

Nope, you can do such stuff:

Code: Select all

TEST_TEXTURE_WIDTH 512
TEST_TEXTURE_HEIGHT 256
TEST_TEXTURE_REAL_WIDTH 480
 sceGuTexImage&#40;0, TEST_TEXTURE_WIDTH, TEST_TEXTURE_HEIGHT, TEST_TEXTURE_REAL_WIDTH, &Bytes&#41;;

Only the width & height arguments must be power of 2, the tbw one can be anything provided it's aligned on 4 bits i think, it's handy to avoid vram wasting.

Doesn't seem to work for me... which could be anything from me screwying up to a change in gimp, to you screwying up ;) ... One way or another I'll look into it more after the compo.

I'll be interested to know what picture and what mode you used to test it. Notice that in order for the alpha channel to be active you'll need to use something like that in your code:

Code: Select all

	sceGuBlendFunc&#40;GU_ADD, GU_SRC_ALPHA, GU_ONE_MINUS_SRC_ALPHA, 0, 0&#41;;
	sceGuEnable&#40;GU_BLEND&#41;;

Post by **Drakonite** » Sun Jun 18, 2006 11:01 pm

hitchhikr wrote:
So it's just my imagination that line 275 is the following? ;)
Code:
Code: Select all
  offset = &#40;&#40;&#40;height - &#40;y + 1&#41;&#41; * width&#41; + x&#41;;
Damn you're right :D
I must have used this for a test purpose but i can't remember which one now, i'll modify that.

That line was for indexed images I believe, you have the same line somewhere else for RGB images I think.. so make sure you catch both (I had to recompile earlier as I missed the indexed one yesterday ;) )

I'll be interested to know what picture and what mode you used to test it. Notice that in order for the alpha channel to be active you'll need to use something like that in your code:
Code: Select all
	sceGuBlendFunc&#40;GU_ADD, GU_SRC_ALPHA, GU_ONE_MINUS_SRC_ALPHA, 0, 0&#41;;
	sceGuEnable&#40;GU_BLEND&#41;;

Hrm... Well I think I was doing things right earlier, but I'll try to play with it more today and see what is going on, as it would be nice to have alpha channels in some of my graphics I'm converting. ;)

hitchhikr · Post by **hitchhikr** » Sun Jun 18, 2006 11:09 pm

Lines: 275, 284, 326 & 375.

I'll remake some tests but i remember i checked the alpha channel stuff thoroughly and it worked fine in every modes.

hitchhikr · Post by **hitchhikr** » Sun Jun 18, 2006 11:18 pm

I forgot to mention, make sure you're using this too:

Code: Select all

	sceGuTexFunc&#40;GU_TFX_REPLACE, GU_TCC_RGBA&#41;;

The default is GU_TCC_RGB i think.

hitchhikr · Post by **hitchhikr** » Mon Jun 19, 2006 1:21 am

Oki i inverted the y coordinates (i think it was because i first tested'em on tga pictures or something) and i fixed a bug in the palette mask constant (wasn't 0xff for > 16 colors palettes).

EDIT: Also it creates xxx_TEXTURE_HEIGHT & xxx_TEXTURE_REAL_HEIGHT constants.

Post by **Drakonite** » Mon Jun 19, 2006 2:28 am

Okay, more bugs...

In the event of of an alpha channel you are reading gimp's data as ABGR, but it seems to be BGRA... and you are using 0xff - A for alpha...
changing those two things I now how correctly working alpha :)

hitchhikr · Post by **hitchhikr** » Mon Jun 19, 2006 2:36 am

Ok, i made a test with this and a texture called test and it worked flawlessly as expected:

Code: Select all

	sceGuClearColor&#40;0xffffffff&#41;;
	sceGuColor&#40;0xffffffff&#41;;

	sceGuClutMode&#40;TEST_PALETTE_FORMAT, 0, TEST_PALETTE_MASK, 0&#41;;
	sceGuClutLoad&#40;TEST_PALETTE_COLORS, &RES_TEXTURE_PALETTE&#41;;
			
	sceGuBlendFunc&#40;GU_ADD, GU_SRC_ALPHA, GU_ONE_MINUS_SRC_ALPHA, 0, 0&#41;;
	sceGuEnable&#40;GU_BLEND&#41;;

	sceGuTexFunc&#40;GU_TFX_REPLACE, GU_TCC_RGBA&#41;;
	sceGuTexMode&#40;TEST_TEXTURE_FORMAT, TEST_TEXTURE_MIPMAPS, 0, TEST_TEXTURE_SWIZZLE&#41;;
	sceGuTexFilter&#40;GU_NEAREST, GU_NEAREST&#41;;

	sceGuTexImage&#40;0, TEST_TEXTURE_WIDTH, TEST_TEXTURE_HEIGHT, TEST_TEXTURE_REAL_WIDTH, &RES_TEXTURE&#41;;
	sceGuEnable&#40;GU_TEXTURE_2D&#41;;

	vertices&#91;0&#93;.tu = 0; vertices&#91;0&#93;.tv = 0;
	vertices&#91;0&#93;.x = 0; vertices&#91;0&#93;.y = 0; vertices&#91;0&#93;.z = 0;
	vertices&#91;1&#93;.tu = TEST_TEXTURE_REAL_WIDTH; vertices&#91;1&#93;.tv = TEST_TEXTURE_REAL_HEIGHT;
	vertices&#91;1&#93;.x = TEST_TEXTURE_REAL_WIDTH; vertices&#91;1&#93;.y = TEST_TEXTURE_REAL_HEIGHT; vertices&#91;1&#93;.z = 0;
	sceGuDrawArray&#40;GU_SPRITES, GU_TEXTURE_32BITF | GU_VERTEX_32BITF | GU_TRANSFORM_2D, 2, 0, vertices&#41;;

	sceGuDisable&#40;GU_TEXTURE_2D&#41;;
	
	sceGuDisable&#40;GU_BLEND&#41;;

Post by **Drakonite** » Mon Jun 19, 2006 2:41 am

Do you mean before or after the change I mentioned?

hitchhikr · Post by **hitchhikr** » Mon Jun 19, 2006 2:42 am

Before your penultimate post.

Post by **Drakonite** » Mon Jun 19, 2006 3:22 am

Hm... k.

FYI I've been using 8888 textures during these test, not paletted textures; that may make a difference...

I've been doing essentially the same as what you've said (though I've done a bunch of other stuff too...) and the original code does not work, while changing the two things I mentioned fixes the issues.

If yours works for you, and mine works for me, I think gimp may have changed how it does things a bit.. As I mentioned I'm using 2.3.8, which is the second to newest dev release... (I don't think they make windows builds of development versions) They changed a lot, so perhaps some of the problems may be caused by changes they've made.

hitchhikr · Post by **hitchhikr** » Mon Jun 19, 2006 3:29 am

If yours works for you, and mine works for me, I think gimp may have changed how it does things a bit.. As I mentioned I'm using 2.3.8, which is the second to newest dev release... (I don't think they make windows builds of development versions) They changed a lot, so perhaps some of the problems may be caused by changes they've made.

That's what i was suspecting.
And it wouldn't surprise me as they always screw everything up from one release to another. Actually there's a 2.3.8 release for windows but even the plugins provided with the distrib don't work. I'll stick to the lastest stable release myself.

EDIT: I tested again with a 8888 texture and it still works perfectly.

hitchhikr · Post by **hitchhikr** » Mon Jun 19, 2006 5:27 am

Actually, you could tell me what code you did modify so maybe i can update the plugin to operate differently under 2.3 & 2.2 ?

hitchhikr · Post by **hitchhikr** » Mon Jun 26, 2006 6:54 pm

v1.0 released:

http://perso.orange.fr/franck.charlet/PSPTex.zip

- Fixed GIMP_RGBA_IMAGE, GIMP_GRAYA_IMAGE & GIMP_INDEXEDA_IMAGE inputs.
- Should now compile correctly under linux & co.

Post by **Oobles** » Tue Jun 27, 2006 8:44 pm

Just to let you know, I've put a mirror up at:

http://ps2dev.org/psp/Tools/PSPTex

I'll post info to news too.

Oobles.

hitchhikr · Post by **hitchhikr** » Tue Jun 27, 2006 8:51 pm

I released a 1.01 version (http://registry.gimp.org/plugin?id=7855).

Also i think you should link it to http://perso.orange.fr/franck.charlet/PSPTex.zip instead of mirroring it.

Post by **Oobles** » Tue Jun 27, 2006 9:01 pm

No problem. Will modify the page to link instead of mirroring.

Oobles.

dot_blank · Post by **dot_blank** » Wed Jun 28, 2006 5:07 am

this is a great plugin under linux thanx alot :)

Findeton · Post by **Findeton** » Sun Sep 03, 2006 12:03 am

Drakonite wrote:I've hacked it up a bit and have gotten it running on the linux version of gimp (developer version 2.3.8)

I've ran into a couple problems though...

The image is upside down (height-(y+1) used instead of y) I know this is often done in windows bmp files, but I don't think this is a good thing for the psp? ... was this intentional or is it a bug?

It seems it's supposed to automatically resize the image to a power of 2 width, however it does not appear to be working... I don't have time to look into why right now unfortunatly.

Also, do you have any intentions of adding support for saving the alpha channel?

I'm also under linux, how can i get it running on the linux version of the gimp?

hitchhikr · Post by **hitchhikr** » Sun Sep 03, 2006 5:55 am

You have to compile it and for this you need the gimp devel package or something.

hitchhikr · Post by **hitchhikr** » Fri Sep 15, 2006 3:58 am

Fixed some issues when compiling under linux.