Phobos icon indicating copy to clipboard operation
Phobos copied to clipboard

[Minor] Make quick save happens after main loop

Open TaranDahl opened this issue 10 months ago • 4 comments

According to what ZivDero told me, saving game inside the main loop may cause corruption. So we need to move it out of the main loop. This needs to be used together with the unmerged yrpp-spawner feature to work properly. It needs to be discussed about how we should interact with spawner. Maybe a flag like CanUseSpawner or sth alike. Open a draft PR to keep this in mind.

TaranDahl avatar Feb 14 '25 09:02 TaranDahl

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

github-actions[bot] avatar Feb 14 '25 09:02 github-actions[bot]

Is there a reason this is a draft?

Metadorius avatar Feb 20 '25 21:02 Metadorius

Is there a reason this is a draft?

Currently, this one can not work correctly without the unmerged AutoSave feature of the spawner. Because the event handling of SaveGame in vanilla is somehow broken. As the spawner rewrote the event handling, I'm not sure what we are going to do with this one. Maybe we need a CanUseSpawner flag to check if the spawner is present, but I dunno how to add it for now.

TaranDahl avatar Feb 21 '25 01:02 TaranDahl

@TaranDahl We have two ways of dealing with that:

  1. Copy the relevant parts from the spawner here so that if any of the spawner and Phobos are present - it works.
  2. Move the quick save command to the spawner.

What do you and others think?

Metadorius avatar Apr 10 '25 21:04 Metadorius

Will be impl in https://github.com/Phobos-developers/Phobos/pull/1790

TaranDahl avatar Jul 30 '25 03:07 TaranDahl