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

Remove disable efb copy #2089

Merged
merged 2 commits into from Feb 24, 2015
Merged

Conversation

degasus
Copy link
Member

@degasus degasus commented Feb 21, 2015

No description provided.

static wxString efb_copy_desc = wxTRANSLATE("Disable emulation of EFB copies.\nThese are often used for post-processing or render-to-texture effects, so while checking this setting may give a minor speedup over EFB to Texture it almost invariably also causes issues.\n\nIf unsure, leave this unchecked.");
static wxString efb_copy_texture_desc = wxTRANSLATE("Store EFB copies in GPU texture objects.\nThis isn't particularly accurate, but it works well enough for most games and gives a great speedup over EFB to RAM.\n\nIf unsure, leave this checked.");
static wxString efb_copy_ram_desc = wxTRANSLATE("Accurately emulate EFB copies.\nNumerous games depend on this for certain graphical effects or gameplay functionality.\nThis is much slower than EFB to Texture.\n\nIf unsure, check EFB to Texture instead.");
static wxString skip_efb_copy_to_ram_desc = wxTRANSLATE("Skip GPU syncing on EFB copys. Causes graphical defects in a small number of other games.\n\nIf unsure, leave this checked.");

This comment was marked as off-topic.

This comment was marked as off-topic.

@neobrain
Copy link
Member

<3

@delroth
Copy link
Member

delroth commented Feb 21, 2015

Can we have JMC and MaJoR report if there are still some games that require this to be playable?

@degasus
Copy link
Member Author

degasus commented Feb 21, 2015

@delroth JMC47 did ask for removing this option: https://code.google.com/p/dolphin-emu/issues/detail?id=8254

@delroth
Copy link
Member

delroth commented Feb 21, 2015

Oh.

@MayImilae
Copy link
Contributor

The last thing that disabling efb copies helped (that I know of) was issue 5920, and that was fixed by TFN. I can't think of any others, and some searching on the wiki didn't find any either.

So I say, let's do this!

@Linktothepast
Copy link
Contributor

Agreed, there is nothing that disabling efb copies really fix. There was also the case of resident evil code veronica in the past but that was properly fixed too.

@Sonicadvance1
Copy link
Contributor

woo

@JMC47
Copy link
Contributor

JMC47 commented Feb 22, 2015

Resident Evil Code Veronica works with EFB2RAM and EFB2Tex. Feel free to merge.

header.bEFBCopyEnable = g_ActiveConfig.bEFBCopyEnable;
header.bCopyEFBToTexture = g_ActiveConfig.bCopyEFBToTexture;
header.bEFBCopyEnable = true;
header.bSkipEfbCopyToRam = g_ActiveConfig.bSkipEfbCopyToRam;

This comment was marked as off-topic.

@Sonicadvance1
Copy link
Contributor

http://hastebin.com/itipekopoc.drps
If you could also remove it from the Android UI using the patch in the hastebin.

@delroth
Copy link
Member

delroth commented Feb 22, 2015

@Sonicadvance1 given that the Android UI still mentions the cache option, I would rather this be all cleaned up in a separate PR.

@Sonicadvance1
Copy link
Contributor

Sounds fine.

delroth added a commit that referenced this pull request Feb 24, 2015
@delroth delroth merged commit f06b110 into dolphin-emu:master Feb 24, 2015
@degasus degasus deleted the remove_disable_efb_copy branch August 9, 2015 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
7 participants