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

[F.Rq.] Expose GPGX' "disable sprite limit" setting #3440

Closed
Sonia-7 opened this issue Nov 5, 2022 · 9 comments
Closed

[F.Rq.] Expose GPGX' "disable sprite limit" setting #3440

Sonia-7 opened this issue Nov 5, 2022 · 9 comments
Labels
Core: BSNES Super Nintendo Entertainment System (SNES) / Super Famicom (SFC) / Super Game Boy (SGB) core Core: Genplus-gx Sega Genesis / Mega Drive core Enhancement For feature requests or possible improvements Repro: Fixed/added in 2.9

Comments

@Sonia-7
Copy link

Sonia-7 commented Nov 5, 2022

It's an enhancement feature that's nice to have for casual playing and such. We already have it for other cores like NES/SMS/PCE, so why not.

Standalone bsnes v115 can already do it natively, so it's just a matter of importing over.

@YoshiRulz YoshiRulz added Enhancement For feature requests or possible improvements Core: BSNES Super Nintendo Entertainment System (SNES) / Super Famicom (SFC) / Super Game Boy (SGB) core Repro: Fixed/added in upstream stable labels Nov 5, 2022
@Morilli
Copy link
Collaborator

Morilli commented Nov 5, 2022

I'm not actually sure what this option does exactly, is it just visual or does it actually affect emulation?
In any case this would be easy to implement; it's one of the configuration options not currently exposed anywhere in the frontend:

@Sonia-7
Copy link
Author

Sonia-7 commented Nov 5, 2022

It's just a visual setting. In the case of SNES it makes possible to display more than 32 sprites per scanline, bypassing the original PPU's limitation.

Such options can be toggled on/off whenever in a core and it's a non-sync setting.

@getCursorsExe
Copy link

To my surprise, this is also an issue in Genplus-gx, the upstream frontend has this option present, while BizHawk doesn't.

@YoshiRulz YoshiRulz added the Core: Genplus-gx Sega Genesis / Mega Drive core label Nov 6, 2022
@Morilli
Copy link
Collaborator

Morilli commented Nov 8, 2022

Do you perhaps know a specific game or test rom that could be used to test this functionality?

@Sonia-7
Copy link
Author

Sonia-7 commented Nov 9, 2022

Do you perhaps know a specific game or test rom that could be used to test this functionality?

Yes. On Alttp's Helmasaur King battle, after you kill him the explosions will normally flicker a little, but with limit disabled they won't.

There's a practice rom of the game floating on the web that's pretty easy to find. Just google "alttp practice rom". It will let you skip straight to that battle through the in-game menu (accessible by pressing R+Start).

@Morilli
Copy link
Collaborator

Morilli commented Nov 9, 2022

Hm, I couldn't figure out how to get the actual boss fight to work, and from a 100% TAS playthrough I also couldn't spot the flickering in the encode (it looked pretty normal to me).

@Sonia-7
Copy link
Author

Sonia-7 commented Nov 10, 2022

I made a save state on e7e587d, using the vanilla USA rom:
helma.zip

This will send you straight to the boss dying. Play it with the limiter enabled, then disabled so you can see the difference. Basically, some of the explosions aren't fully rendered (parts of the graphics are missing).

@Morilli
Copy link
Collaborator

Morilli commented Nov 10, 2022

Have resolved the bsnes part of this with c21fedc. Wanna keep this open for genplus-gx specifically @YoshiRulz ?

@YoshiRulz YoshiRulz removed the Core: BSNES Super Nintendo Entertainment System (SNES) / Super Famicom (SFC) / Super Game Boy (SGB) core label Nov 10, 2022
@YoshiRulz YoshiRulz changed the title "No sprite limit" option missing from bsnes v115 [F.Rq.] Expose GPGX' "disable sprite limit" setting Nov 10, 2022
@Sonia-7
Copy link
Author

Sonia-7 commented Nov 22, 2022

GPGX is currently the only major older system missing this feature. Do you think you can expose for it as well, @Morilli?

Morilli added a commit to Morilli/BizHawk that referenced this issue Nov 22, 2022
- resolves TASEmulators#3440
- includes manually cherry-picked e0ef0902e96bd9d71cb49c64505e755007e7452c
@YoshiRulz YoshiRulz added Core: BSNES Super Nintendo Entertainment System (SNES) / Super Famicom (SFC) / Super Game Boy (SGB) core Repro: Fixed/added in 2.9 and removed Repro: Fixed/added in upstream stable labels Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Core: BSNES Super Nintendo Entertainment System (SNES) / Super Famicom (SFC) / Super Game Boy (SGB) core Core: Genplus-gx Sega Genesis / Mega Drive core Enhancement For feature requests or possible improvements Repro: Fixed/added in 2.9
Projects
None yet
Development

No branches or pull requests

4 participants