|
Re: The MiniGL thread |
Posted on: 2019/3/26 13:38
#141 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2015/6/11 9:51 From Cologne
Posts: 433
|
@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) {
|
|
|
Re: The MiniGL thread |
Posted on: 2019/3/26 13:51
#142 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
@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 |
||
|
Re: The MiniGL thread |
Posted on: 2019/3/28 10:16
#143 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2015/6/11 9:51 From Cologne
Posts: 433
|
@Raziel
Did it work? |
|
|
Re: The MiniGL thread |
Posted on: 2019/3/28 11:12
#144 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
@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 |
||
|
Re: The MiniGL thread |
Posted on: 2019/3/28 17:08
#145 |
---|---|---|
Just can't stay away
![]() ![]() Joined:
2007/7/14 21:30 From Lothric
Posts: 1198
|
@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. |
|
|
Re: The MiniGL thread |
Posted on: 2019/3/28 17:33
#146 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
@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 |
||
|
Re: The MiniGL thread |
Posted on: 2019/3/28 17:56
#147 |
---|---|---|
Just can't stay away
![]() ![]() Joined:
2007/7/14 21:30 From Lothric
Posts: 1198
|
@Raziel
Maybe you can request the main developers to test with power-of-two textures? |
|
|
Re: The MiniGL thread |
Posted on: 2019/3/29 7:50
#148 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2015/6/11 9:51 From Cologne
Posts: 433
|
@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;
- 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. |
|
|
Re: The MiniGL thread |
Posted on: 2019/3/29 18:36
#149 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
@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 |
||
|
Re: The MiniGL thread |
Posted on: 2019/3/29 19:19
#150 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2015/6/11 9:51 From Cologne
Posts: 433
|
@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. |
|
|
Re: The MiniGL thread |
Posted on: 2019/3/29 19:33
#151 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
@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 |
||
|
Re: The MiniGL thread |
Posted on: 2019/3/31 17:31
#152 |
---|---|---|
Just can't stay away
![]() ![]() Joined:
2007/7/14 21:30 From Lothric
Posts: 1198
|
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:
I can make a BZ ticket if feature seems worth adding. |
|
|
Re: The MiniGL thread |
Posted on: 2019/4/10 7:04
#153 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2007/1/6 5:59 Posts: 350
|
What is current state of this?
I installed minigl from post 92. It seems to have some problems with shadows at least with Shogo. |
|
|
Re: The MiniGL thread |
Posted on: 2019/4/10 7:17
#154 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2015/6/11 9:51 From Cologne
Posts: 433
|
@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. |
|
|
Re: The MiniGL thread |
Posted on: 2019/4/10 9:10
#155 |
---|---|---|
Not too shy to talk
![]() ![]() Joined:
2007/1/6 5:59 Posts: 350
|
@Daytona675x
Are you interested results? |
|
|
Re: The MiniGL thread |
Posted on: 2019/4/10 17:46
#156 |
---|---|---|
Just can't stay away
![]() ![]() Joined:
2007/7/14 21:30 From Lothric
Posts: 1198
|
@utri007
Does the pope have a balcony? |
|
|
Re: The MiniGL thread |
Posted on: 2019/5/25 17:55
#157 |
---|---|---|
Just can't stay away
![]() ![]() Joined:
2007/7/14 21:30 From Lothric
Posts: 1198
|
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 |
|
|
Re: The MiniGL thread |
Posted on: 2019/5/25 18:03
#158 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
|
|
_________________
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 |
||
|
Re: The MiniGL thread |
Posted on: 2019/5/25 18:08
#159 |
---|---|---|
Just can't stay away
![]() ![]() Joined:
2007/7/14 21:30 From Lothric
Posts: 1198
|
@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. |
|
|
Re: The MiniGL thread |
Posted on: 2019/5/25 18:15
#160 |
---|---|---|
Home away from home
![]() ![]() Joined:
2006/11/26 21:45 From a dying planet
Posts: 3934
|
@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 |
||