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

Gameini updates. #397

Merged
merged 3 commits into from May 31, 2014
Merged

Gameini updates. #397

merged 3 commits into from May 31, 2014

Conversation

Linktothepast
Copy link
Contributor

Addition of most Virtual Console NES and Master System games to the database.
Fixes issues 7311 and 7272.

@delroth
Copy link
Member

delroth commented May 23, 2014

@JMC47 @MaJoRoesch care to review that?

@JMC47
Copy link
Contributor

JMC47 commented May 23, 2014

Honestly looks pretty good. I reviewed the games I know that could use EFB2Texture, and it looks like the INIs got 'em. There may be some I'm missing since there are so many, but it seems good to me.

Is that all the NES games in all regions?

@Linktothepast
Copy link
Contributor Author

Most of them, just a few are missing, same for Master System.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

For Virtual Console NES games, scaled EFB copy being off no longer matters. They do need 1x IR though.

@Linktothepast
Copy link
Contributor Author

I've set it to integral values since they seem to be working well.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

This is not true for most games. 3X IR kills graphics in Castlevania, for instance.

@MayImilae
Copy link
Contributor

Confirming what JMC is saying. EFBscale isn't a problem, but anything over 1x Native screws up most NES games, even integral values like 2x native.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

Scaled EFB copy on does cause problems in Zelda 1 at 2x IR, while turning it off does work for 2x IR at least.

@MayImilae
Copy link
Contributor

Yea well, just leave it on for that one then. shrug

@Linktothepast
Copy link
Contributor Author

Guys can you post a screen shot of the difference? I tried castlevania 3x and it seems fine here.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

Absolutely, sorry for being lazy about it.

1x: https://dl.dropboxusercontent.com/u/484730/FEQE01-5.png
3x: https://dl.dropboxusercontent.com/u/484730/FEQE01-6.png

Edit: It is known that NES games behave differently on NVIDIA and AMD graphics cards. I have NVIDIA. Also happens if you have Widescreen on, which I turned off 'cause eww stretching.

@MayImilae
Copy link
Contributor

Could it be an NTSC/PAL difference?

@Linktothepast
Copy link
Contributor Author

I am not getting that issue jmc so we need to track it down a bit. Did you use a PAL version? What were the rest of your settings? Also do you guys have an AMD card?

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

Of relevant settings: EFB Copies is to Ram, Texture Cache is safe, XFB disabled (needed for every game anyway,) no Anti-Aliasing, no Anistrophic Filtering, no other enhancements. No hacks enabled and nothing checked in General either.

I do have an AMD card, but putting it into my computer is a bit of a hassle. FWIW, this also happen on my laptop. GTX 760 on my main box, GTX 560m on my Laptop. Both have intel processors, one is Windows, other is Linux.

@Linktothepast
Copy link
Contributor Author

Right, i 've found it, seems an opengl issue, direct 3d works fine. So enforce 1x anyway to cover every case?

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

I have a question: Does D3D work without issue for you? What graphics card are you running?

I ask this because D3D does not work properly with Virtual Console games for me. Constant flickering/glitches.

@Linktothepast
Copy link
Contributor Author

Yes it works just fine and i have a 560 ti with latest drivers.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

Okay, let me update drivers; the whole reason I haven't updated is due to the OGL crash. If I can confirm it's a driver issue and check it in my games, then I think letting scaled EFB on is fine.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

The more I think about it, there's literally no reason to up the IR in NES VC games, so, I'd actually rather have it forced to 1x IR. Considering OSX and Linux only have OGL, it may be a better idea.

@Linktothepast
Copy link
Contributor Author

OK, we have a consensus then in enforcing 1xIR?

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

I think it's fine. There's no noticeable benefit to upping the IR, right? If there is, maybe there'd be an argument against it, but I can't think of anything that would matter for old 2D games.

@Linktothepast
Copy link
Contributor Author

It makes graphics sharper as if they passed from a sharpening filter. Nothing more i can think.

@JMC47
Copy link
Contributor

JMC47 commented May 24, 2014

On latest drivers I still can't get D3D to work.

@MayImilae
Copy link
Contributor

Enforcing 1x IR on NES games is fine with me. While higher IR is sharper, if it doesn't work it doesn't work, and the vast majority don't.

Various changes.
Change Nes games efb scaling to 1x. Also fix issue 7314.
@Linktothepast
Copy link
Contributor Author

Ok it is done. I also fixed issue 7314 in the process.

Updates for F-Zero X (n64), Yoshi's story (n64) and Curious George (gc).
Fixes issue 7333.
@Linktothepast
Copy link
Contributor Author

I also fixed issue 7333 along with some small updates.

@JMC47
Copy link
Contributor

JMC47 commented May 31, 2014

I think we're all in agreement with this now? I think this is good to merge.

@MayImilae
Copy link
Contributor

Ahh, added the ZTP hack patch. JMC47 you've tested it, sure it should go in?

@JMC47
Copy link
Contributor

JMC47 commented May 31, 2014

Skid and I have tested it. It has to be manually enabled anyway.

@MayImilae
Copy link
Contributor

lgtm then.

@JMC47
Copy link
Contributor

JMC47 commented May 31, 2014

LGTM

neobrain added a commit that referenced this pull request May 31, 2014
@neobrain neobrain merged commit 31eedb2 into dolphin-emu:master May 31, 2014
@MayImilae
Copy link
Contributor

You can see the INIs that are affected by looking at the files changed by the commit.

@takeshineale128
Copy link

gameini - fix pal NES games get black Screen and garble mess.

@JMC47
Copy link
Contributor

JMC47 commented Jul 19, 2014

Stop posting on the github commits unrelated to your problem. Use OpenGL, EFB2RAM, and safe texture cache on NVIDIA cards. It's not Dolphin's bug!

@takeshineale128
Copy link

on osx (nvidia 320m) its on OpenGL, efb2ram & safe texture but the graphic is in a mess.

@JMC47
Copy link
Contributor

JMC47 commented Jul 19, 2014

That is driver problems, not the emulator. Now please stop posting on unrelated pulls.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
7 participants