Skip to content

GUI/SaveState: Progress Dialog On Save State Creation, Saving Fixes And Speed Improvement #15345

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

Merged
merged 6 commits into from
Mar 24, 2024

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Mar 24, 2024

Savestate creation is relatively long due to compression, it makes some users think savestates do not work or that RPCS3 crashed. So I propose for now a progress dialog for savestate creation which shows how much time passed and how many bytes were written.
The percentage shown is not accurate and is an estimation of the amount of bytes written, never actually reaching 100% before completion in order to simplify the implementation. 300MB = 50%, 600MB = 75%, 1200MB = 87.5%, most games will quit saving at around 50%.

image

  • I also fixed 2 bugs in savestates, one which causes saving crash and the other caused a loading crash.
  • Compression level has been reduced to 8 for faster saving at few more MBs.

@elad335 elad335 added the Savestates Anything that involves savestates label Mar 24, 2024
@elad335 elad335 force-pushed the get-rid branch 2 times, most recently from fee5f60 to 25b6235 Compare March 24, 2024 12:32
@elad335 elad335 force-pushed the get-rid branch 2 times, most recently from db3feb1 to 1611d9a Compare March 24, 2024 13:05
@elad335 elad335 changed the title GUI: Progress Dialog On Save State Creation GUI: Progress Dialog On Save State Creation, Savestate Fixes Mar 24, 2024
@elad335 elad335 changed the title GUI: Progress Dialog On Save State Creation, Savestate Fixes GUI/SaveState: Progress Dialog On Save State Creation, Saving Fixes And Speed Improvement Mar 24, 2024
@elad335 elad335 merged commit 3393d13 into RPCS3:master Mar 24, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix GUI Savestates Anything that involves savestates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant