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

Rewind is now limited / missing many features #2300

Closed
Sonia-7 opened this issue Aug 13, 2020 · 6 comments
Closed

Rewind is now limited / missing many features #2300

Sonia-7 opened this issue Aug 13, 2020 · 6 comments
Assignees
Labels
App: EmuHawk Relating to EmuHawk frontend Enhancement For feature requests or possible improvements Repro: Fixed/added in 2.5

Comments

@Sonia-7
Copy link

Sonia-7 commented Aug 13, 2020

In recent builds, the rewind window became overly simplistic, with many features missing. You can no longer set the rewind frequency every x frames, no "Use disk for buffer instead of RAM", etc, and the biggest issue is that now the Max buffer size is only 1024MB. Even with "use compression" checked, this is hardly enough to rewind even a few minutes in stuff like SNES games. Meanwhile in previous versions I could rewind up to over 1 hour.

Here's a screenshot comparison between the old and new window:
https://i.imgur.com/fhg2baP.png
https://i.imgur.com/EETDAYw.png

What explains such a massive downgrade?

@RetroEdit
Copy link
Contributor

The rewind implementation was redone in the newest dev build, with something that's been called 'zwinder'. The intention there was to increase the efficiency of savestates for waterboxed cores.

@nattthebear is probably the best equipped to answer your questions.

@Sonia-7
Copy link
Author

Sonia-7 commented Aug 13, 2020

Can't at least the max buffer size get increased? Maybe up to 8192MB or so would already make a difference. 1024MB is too little.

@nattthebear
Copy link
Contributor

Frequency is set by determining the desired frame length together with the buffer size and the core's state size. Max buffer size larger than 1GB is something I've been thinking about; is there much interest in it?

@Sonia-7
Copy link
Author

Sonia-7 commented Aug 13, 2020

I don't know where to change the state size anymore. Could you tell where it is?

And well, higher max buffer size is certainly something I'd be interested in. I mean, most machines nowadays can have 32GB+ of RAM (mine has 16GB), so personally I don't see why having such a small limit.

@nattthebear
Copy link
Contributor

By state size I mean the core's actual savestate size, which cannot be changed (except by turning on compression, sort of.) So for instance, if your buffer is 512MB large, and states are 300KB large, then the buffer can hold about 1747 states. If you ask for a frame length of 10000, that means the frequency will be about one state every 5-6 frames.

This behavior was needed to better support cores with variable savestate sizes.

@nattthebear nattthebear self-assigned this Aug 13, 2020
@nattthebear
Copy link
Contributor

I did a bit of rearchitecting in the rewinder to support larger buffers. The UI now caps you at 32GiB buffers (although if you manually edited your config, I suppose you could go larger.)

Tell me how it works out for you, and please reopen this issue or create another one if there are any further gaps in functionality.

@YoshiRulz YoshiRulz added App: EmuHawk Relating to EmuHawk frontend Enhancement For feature requests or possible improvements Repro: Fixed/added in 2.5 labels Aug 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: EmuHawk Relating to EmuHawk frontend Enhancement For feature requests or possible improvements Repro: Fixed/added in 2.5
Projects
None yet
Development

No branches or pull requests

4 participants