Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GLideN64 Regression #1646

Closed
nlburgin opened this issue Aug 23, 2019 · 10 comments
Closed

GLideN64 Regression #1646

nlburgin opened this issue Aug 23, 2019 · 10 comments
Assignees
Labels

Comments

@nlburgin
Copy link

nlburgin commented Aug 23, 2019

There's a strange glitch happening in GLideN64 on Mario Golf in Bizhawk 2.3.1 and 2.3.2 (as well as latest interim build)

It looks like this.

Mario Golf (U)  ! 2019-08-22 17 59 19

The texture is blacked out on the fairway, the tee box, and the green.

However, on 2.3.0, it looks like this:

Mario Golf (U)  ! 2019-08-22 18 02 22

which is clearly a lot more normal. The sync settings were exactly the same between these two pictures. In fact, this result can also be achieved on 2.3.2 if you simply swap out its copy of mupen64plus-video-GLideN64.dll with the older one that came with 2.3.0.

I initially assumed this was an upstream regression, but when I tested the latest upstream GLideN64 with the latest upstream Mupen64Plus, the glitch did not occur. So it seems like it has to be a Bizhawk issue, then.

Interestingly, the upstream DLL did not seem to be compatible with BizHawk. As I mentioned it worked fine with regular M64+, but when I tried it with Bizhawk, it did this:

Mario Golf (U)  ! 2019-08-22 18 15 22

Which is definitely worse. Although from what little I can make out through the complete scramblage, at least the texture glitch appears to be absent 😝

Incidentally I was on IRC (as "Question48") and someone suggested removing all the Mario Golf related lines from gamedb_n64.txt, but that had no effect. So, it seems like the answer isn't in there.

@zeromus
Copy link
Contributor

zeromus commented Aug 23, 2019

bizhawk is not using the latest upstream gliden64, so it's not proof of anything.
the only interesting proof of anything would be demonstrating a gilden64 version that had that bug on mupen. ideally, the version we last merged.

additionally it would help if someone merged the latest gliden64 code, since you did test that version on mupen. it's possible it will work again in bizhawk

@nlburgin
Copy link
Author

nlburgin commented Aug 23, 2019

ideally, the version we last merged.

Would that be this?

Unfortunately, that commit is over six months old, so its appveyor artifacts got automatically deleted. I'm not confident trying to build it from source myself ☹️

@zeromus
Copy link
Contributor

zeromus commented Aug 23, 2019

no, that's the master branch. the last version we merged is from TASVideos/GLideN64:Branch_bizhawk

edit: link --yoshi

@nlburgin
Copy link
Author

Oh. Well in that case it looks like the latest merge was Public Release 4.0, which is what I tested, and what I meant when I mentioned the latest upstream Gliden. I wasn't using an interim dev build of gliden, just the most recent release.

So it looks like the version that was merged last was the one I tested after all.

@YoshiRulz YoshiRulz added Core: Mupen64Plus Nintendo 64 (N64) core Repro: Fixed/added as of 2.3/2.3.1/2.3.2 (2.3.3 has its own label) Repro: Regression from ??? I don't remember when exactly this used to work, but I swear it did Repro: Affects 2.3/2.3.1/2.3.2 (2.3.3 has its own label) labels Aug 23, 2019
@nlburgin
Copy link
Author

It might be worth noting that I could only find a 32-bit build of upstream M64+ for Windows, so when I tested upstream Gliden with it I also used the 32-bit version library.

Whereas I'm pretty sure everything to do with Bizhawk was 64-bit.

I wouldn't think that would be related to anything like this, but IDK.

@nlburgin
Copy link
Author

It also might be worth mentioning that in all cases I was using OpenGL 3.3. core profile.

I think gliden is designed do things somewhat differently internally depending on what OpenGL version it detects. So I'm not 100% sure if the bug would be reproducible in an OpenGL 4.x environment due to different code paths possibly being involved.

IDRK, it just seemed worth mentioning.

@vadosnaprimer
Copy link
Contributor

We've seen bugs that were only caused by the way we forked gliden, it's probably one of those.

@nlburgin nlburgin mentioned this issue Sep 3, 2019
@YoshiRulz YoshiRulz added Repro: Regression from 2.3 Repro: Affects 2.3/2.3.1/2.3.2 (2.3.3 has its own label) and removed Repro: Affects 2.3/2.3.1/2.3.2 (2.3.3 has its own label) Repro: Fixed/added as of 2.3/2.3.1/2.3.2 (2.3.3 has its own label) Repro: Regression from ??? I don't remember when exactly this used to work, but I swear it did labels Dec 25, 2019
@vadosnaprimer vadosnaprimer self-assigned this Jan 12, 2020
@vadosnaprimer
Copy link
Contributor

vadosnaprimer commented Feb 4, 2020

Test latest master https://ci.appveyor.com/project/zeromus/bizhawk-udexo/build/artifacts
Might not help at all, but there's not a lot we can do with our 7 year old core.

@YoshiRulz YoshiRulz added the Repro: Patch pending Potentially fixed in dev build, see readme for download label Feb 4, 2020
@nlburgin
Copy link
Author

nlburgin commented Mar 20, 2020

I couldn't even get the dev build to run (see #1886)

However, I can confirm that my workaround of simply copy-pasting the mupen64plus-video-GLideN64.dll from 2.3.0 to the newer version still works in the latest stable release 2.4.0

@vadosnaprimer
Copy link
Contributor

Main issue should be #1718

@YoshiRulz YoshiRulz removed the Repro: Patch pending Potentially fixed in dev build, see readme for download label Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants