Login
Username:

Password:

Remember me



Lost Password?

Register now!

Sections

Who's Online
119 user(s) are online (62 user(s) are browsing Forums)

Members: 0
Guests: 119

more...

Headlines

 
  Register To Post  

« 1 ... 5 6 7 (8) 9 10 11 »
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
@Raziel
Okay, as far as I can see the following happens:

1. if there's an NPOT texture incoming, then the game creates an empty texture with the next power of two edge size. So e.g. a 200x100 texture would become 256x128. So far so good.
2. but the game doesn't scale the texture data accordingly but simply puts the tex-data unscaled at position 0,0.
3. and because the texture was created as an empty texturem, a huge part of it is actually undefined!

Now take all that and add a slightly false texture-coordinate generation, voila: you have a randomly colored (fully black or white are eventually most likely depending on driver) line like the one you see.
And yes, the wrap-mode doesn't really have the expected impact here, because we are in mid of the texture data, not at the edges (although a GL_REPEAT most likely makes things even worse here).

You could now try to locate the texture coordinate generation code for that sky-cube and fix it.
Or as a simple slow and dirty workaround you could try adding the following code after line 96 - and eventually improve it later

if(!OpenGLContext.NPOTSupported) {
    if(
rect.left==&& rect.top==&& rect.right==surface->&& rect.bottom==surface->&& (internalWidth!=width || internalHeight!=height)) {
        
glTexSubImage2D(GL_TEXTURE_2D,0,1,1,surface->w,surface->h,internalFormat,sourceFormat,surface->getPixels());
        
glTexSubImage2D(GL_TEXTURE_2D,0,1,0,surface->w,1,internalFormat,sourceFormat,surface->getPixels());
        
glTexSubImage2D(GL_TEXTURE_2D,0,0,1,surface->w,surface->h,internalFormat,sourceFormat,surface->getPixels());
    }
}

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Daytona675x

Thank you, i'll try that later

Funny you should assume i could "improve" anything, i couldn't even find the culprit in the first place if my life depended on it

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
@Raziel
Did it work?

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Daytona675x

Sorry, i didn't want to draw any more of your time away from the important stuff.

Yes, that fix makes the visible borders go away, but it also slows down the game a lot.

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top
Re: The MiniGL thread
Just can't stay away
Just can't stay away


See User information
@Raziel

Do you have a Linux system? To me it sounds like issue would be reproducible on other systems as well. If I remember correctly it's possible to configure ScummVM's OpenGL context (maybe ResidualVM too) in such a way that it's forced to use POT-textures.

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Capehill

Unfortuntely not, but i could provide the build and the data files for testing

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top
Re: The MiniGL thread
Just can't stay away
Just can't stay away


See User information
@Raziel

Maybe you can request the main developers to test with power-of-two textures?

Go to top
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
@Raziel
Quote:
Sorry, i didn't want to draw any more of your time away from the important stuff.



Quote:
Yes, that fix makes the visible borders go away, but it also slows down the game a lot.

Please do this:

- remove the slow texture-filler I gave you
- check file myst3/gfx_opengl.cpp
- line 263 and 264 contain the wrong coordinate calculations
- actually the whole source is full of them, but for your problem those are the relevant lines as far as I can see
- replace those two line by those here:
const float uv_mod=1.0f;
const 
float w = (glTexture->width-uv_mod) / (float) glTexture->internalWidth;
const 
float h = (glTexture->height-uv_mod) / (float) glTexture->internalHeight;

- maybe an uv_mod of 0.5 yields better results (it should but sometimes I mess the details of those coordinates systems up myself )

@Capehill
Quote:
Do you have a Linux system? To me it sounds like issue would be reproducible on other systems as well.

Yes, it should, it's a typical classic OpenGL pitfall. However, most systems support NPOT, that's why it probably wasn't noticed earlier.

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Daytona675x

Thank you, that fixed it too.

Using 0.5f instead of 1.0f gives me an instant lockup of the whole system (when i start the game).
Actually so bad that it takes at least 5-7 minutes to bring it up again.

Nothing on console or serial.
Seems to be locking up the gfx hardware as numerous (unsuccessful) tries to boot the system up again are stopping at a message that there was no VGA device found (on serial).

I need to turn the whole thing off, let it sit there (power plug out the socket) for some minutes to be able to restart it again.

Not sure if my gfx card is dying (otoh i know this deep lockups from the very first time i got the X1000).

It still is slower than without the patch, but faster than the patch before

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
@Raziel
in this universe it is exactly 100% absolutely totally for sure and without any possible fraction of a theoretical doubt impossible that this change

- has any impact on stability, neither positive nor negative
- has any impact on performance, neither positive nor negative
- and in both cases the value of the constant, be it 1.0f or 0.5f, cannot produce any other possible difference than a tiny difference in the filtered output at the cube's problematic edges, no more no less

Something else must be broken on your end. It can not possibly have anything to do with this change.

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Daytona675x

I believe you...

There is a slim chance that i caught that dreaded "locking up on me on closing a SDL window" bug exactly five times in a row.

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top
Re: The MiniGL thread
Just can't stay away
Just can't stay away


See User information
Jump from SDL2 topic: http://www.amigans.net/modules/xforum ... id=113935#forumpost113935

It seems that MiniGL doesn't support glGetIntegerv(GL_DOUBLEBUFFER). It would be seemingly simple to add, something like:

Quote:

Index: src/state_get.c
===================================================================
--- src/state_get.c (revision 541)
+++ src/state_get.c (working copy)
@@ -728,6 +728,10 @@
}
#endif

+ case GL_DOUBLEBUFFER:
+ READ_BOOL1(context->NumBuffers > 1);
+ break;
+
/* -------------- ONLY ENABLES BEYOND THIS POINT ---------------------- */
/* FIXME: Add other state not required for second milestone */


I can make a BZ ticket if feature seems worth adding.

Go to top
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
What is current state of this?

I installed minigl from post 92. It seems to have some problems with shadows at least with Shogo.

Go to top
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
@utri007
The state is wip, other things to do.
Simply use the version from os4depot for now and eventually update when you hear that we're done.

Go to top
Re: The MiniGL thread
Not too shy to talk
Not too shy to talk


See User information
@Daytona675x

Are you interested results?

Go to top
Re: The MiniGL thread
Just can't stay away
Just can't stay away


See User information
@utri007

Does the pope have a balcony?

Go to top
Re: The MiniGL thread
Just can't stay away
Just can't stay away


See User information
As Raziel reported earlier in some SDL thread, he has a problem when game's Image C++ namespace collides with some Amiga definition. It can be reproduced with an example like:

-- clip --

#include "GL/gl.h"

namespace Image { };

int main(void)
{
return 0;
}

-- clip --

Defining __USE_AMIGAOS_NAMESPACE__ gives only more compilation error.

The issue happens because "mgl/context.h" includes "intuition/intuition.h". As far as I can see, including "intuition.h" is unnecessary. MiniGL compiles without it. I suppose it has been included because of Screen/Window pointers but on the other hand, why? Context.h doesn't need to know how Screen/Window fields look like.

@Raziel: as a workaround, you can comment out that include line in "mgl/context.h" and try again your project.

References:

http://www.hyperion-entertainment.com ... -kc/include/mgl/context.h
http://www.amigans.net/modules/xforum ... id=113800#forumpost113800

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Capehill

Will try.

Do the SDL change need to be in place too?

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top
Re: The MiniGL thread
Just can't stay away
Just can't stay away


See User information
@Raziel

Only in case of SDL1 and if SDL_syswm.h is included - you can see the include tree in compilation error.

So we have 2 issues: one caused by SDL1 and other by MiniGL.

Go to top
Re: The MiniGL thread
Home away from home
Home away from home


See User information
@Capehill

Ah, understood, thank you for the explanation

People are dying.
Entire ecosystems are collapsing.
We are in the beginning of a mass extinction.
And all you can talk about is money and fairytales of eternal economic growth.
How dare you!
– Greta Thunberg
Go to top

  Register To Post
« 1 ... 5 6 7 (8) 9 10 11 »

 




Currently Active Users Viewing This Thread: 1 ( 0 members and 1 Anonymous Users )




Powered by XOOPS 2.0 © 2001-2023 The XOOPS Project