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

Do not try to load strided efb copies or ones with wrong dimensions #3165

Merged

Conversation

mimimi085181
Copy link
Contributor

And also keep those efb copies, so they can be used by a future Load() or by partial texture updates.

This fixes some issues in: Sonic Riders: Zero Gravity, Mario Kart Wii and Sonic the Fighters (inside Sonic Gems Collection).

This hopefully allows to use partial texture updates with strided efb copies that start at the same offset as the loaded texture
@mimimi085181 mimimi085181 force-pushed the partial-texture-updates-same-offset branch from 96e7aa8 to af28934 Compare October 14, 2015 22:46
@mimimi085181 mimimi085181 changed the title Do not delete strided efb copies on load Do not load strided efb copies or ones with wrong dimensions Oct 14, 2015
@mimimi085181 mimimi085181 changed the title Do not load strided efb copies or ones with wrong dimensions Do not try to load strided efb copies or ones with wrong dimensions Oct 14, 2015
@dolphin-emu-bot
Copy link
Contributor

FifoCI detected that this change impacts graphical rendering. Here are the behavior differences detected by the system:

  • inverted-depth-range on ogl-lin-mesa: diff

automated-fifoci-reporter

@degasus
Copy link
Member

degasus commented Oct 15, 2015

LGTM

@DonelBueno
Copy link

What issue does this fix in Mario Kart Wii? Would it be the lens flare effect with "EFB Copies to Texture Only"?

@mimimi085181
Copy link
Contributor Author

Sorry no, it's should be this issue: https://bugs.dolphin-emu.org/issues/9027
Everything being darker and strange dark textures in Mario Kart should be fixed. But i'm not sure about drift and trick textures, i don't even know how that is supposed to look.

degasus added a commit that referenced this pull request Oct 16, 2015
…me-offset

Do not try to load strided efb copies or ones with wrong dimensions
@degasus degasus merged commit d12b026 into dolphin-emu:master Oct 16, 2015
@wtonberrios
Copy link

Español

Hola, desde esta compilacion detecte un error en Donkey Kong Country Returns, el personaje de donkey kong desaparece y aparece a cada rato, al desactivar "Copia de EFB a escala" vuelve a funcionar sin problemas, pero con menor calidad en las texturas del donkey kong, la compilacion dolphin-master-4.0-8008 funciona bien en comparacion con esta version dolphin-master-4.0-8010.

PD: se mantiene este error hasta la compilacion dolphin-master-4.0-8054.

Solo informaba, saludos
Wton.

English (Google traslator) (si no se entiende bien, alguna persona podría traducirla por favor, muchas gracias)

Hello, since this compilation detects an error in Donkey Kong Country Returns, Donkey Kong character disappears and all the time is displayed, deactivate "Copying EFB scale" run smoothly again, but with lower quality textures donkey kong, compilation dolphin-master-4.0-8008 works well with this version compared dolphin-master-4.0-8010.

PD: This error is held until compilation dolphin-master-4.0-8054.

Only reported, Greetings
Wton.

http://www.servimg.com/view/11866238/1205
sf8e01-1v3

@7oxicshadow
Copy link

Just to confirm, I also see the same issue as wtonberrios on Linux.
This commit breaks DKCR. Character and enemy models either flicker or just dont show at all.

Fedora 22
Nvidia proprietary (355.11)
GTX 770
i7 @ 3.6Ghz
6 GB ram

mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Nov 17, 2015
As long as an efb copy overlaps a normal texture, copy the overlapping part from the efb copy into the normal texture. Before the pr, this was restricted to efb copies that are completely within the normal texture.

This is supposed to fix the invisible character in Donkey Kong Returns, which was caused by dolphin-emu#3165, because it tries to load smaller textures than the created efb copies.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Nov 17, 2015
As long as an efb copy overlaps a normal texture, copy the overlapping part from the efb copy into the normal texture. Before the pr, this was restricted to efb copies that are completely within the normal texture.

This is supposed to fix the invisible character in Donkey Kong Returns, which was caused by dolphin-emu#3165, because it tries to load smaller textures than the created efb copies.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Nov 18, 2015
As long as an efb copy overlaps a normal texture, copy the overlapping part from the efb copy into the normal texture. Before the pr, this was restricted to efb copies that are completely within the normal texture.

This is supposed to fix the invisible character in Donkey Kong Returns, which was caused by dolphin-emu#3165, because it tries to load smaller textures than the created efb copies.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Nov 18, 2015
As long as an efb copy overlaps a normal texture, copy the overlapping part from the efb copy into the normal texture. Before the pr, this was restricted to efb copies that are completely within the normal texture.

This is supposed to fix the invisible character in Donkey Kong Returns, which was caused by dolphin-emu#3165, because it tries to load smaller textures than the created efb copies.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Nov 21, 2015
As long as an efb copy overlaps a normal texture, copy the overlapping part from the efb copy into the normal texture. Before the pr, this was restricted to efb copies that are completely within the normal texture.

This is supposed to fix the invisible character in Donkey Kong Returns, which was caused by dolphin-emu#3165, because it tries to load smaller textures than the created efb copies.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Mar 26, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Mar 26, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Mar 26, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Mar 26, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Mar 26, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Apr 2, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Apr 9, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
mimimi085181 added a commit to mimimi085181/dolphin that referenced this pull request Apr 9, 2016
This pr fixes another regression that happened after pr dolphin-emu#3165 was merged. The shadows in mario baseball should now work again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants