LiveSplit icon indicating copy to clipboard operation
LiveSplit copied to clipboard

Issue #952 - Added hibernate and resume run

Open goodtrailer opened this issue 5 years ago • 9 comments

#952

Adds:

  • Hibernating (saving) a paused run to a .lsr (LiveSplit Run) file that is formatted in XML
  • Loading from a .lsr as long as the timer isn't running. After loading the run, the timer is still paused, it doesn't immediately resume.
  • Appropriate right click menu buttons (under "Controls") and hotkey settings (unbound by default)

Notes:

  • Disabled if autosplitter is active
  • Exception handling might be a little spotty
  • Possible confusion in method/variable naming, because while the word "run" is usually used to mean a specific speedrun, in the code the Run class is used to store all of the past runs, metadata, game name, category, etc. For example, the new TimerModel.LoadRun() isn't used to load from an entire Run class, just some specific variables.

goodtrailer avatar Apr 20 '20 04:04 goodtrailer

Maintainers, this feature has been requested several times in issues, so why has it been ignored/overlooked?

kkaja123 avatar Dec 27 '20 15:12 kkaja123

Yeah I'd prefer it as well if we don't have a new file format for this.

CryZe avatar Dec 27 '20 18:12 CryZe

I agree that in-progress runs should be hibernated into the splits file instead of a separate .lsr, but I'm not sure that's possible without making changes to livesplit-core, since RunFactory relies on the extern Run_parse. And since modifying livesplit-core is probably the proper way to go about this, most of what I've written here seems to be the wrong approach. (Maybe I'll see if I can make those changes to livesplit-core, but I'm pretty unfamiliar with Rust.)

goodtrailer avatar Dec 30 '20 02:12 goodtrailer

@goodtrailer I'm really eager to get this feature merged into the main project. I've been using your code for a few days (and fixed one issue related the date/time format, because I'm in Australia). I was just wondering if you're planning on making the suggested changes to this pull request? If not, I'd be happy to make the changes myself, and put up a new pull request.

benshemmeld avatar Jan 08 '21 02:01 benshemmeld

@benshemmeld I was planning on looking into livesplit-core when I found the time, but if you do find a way to make the code save to the splits file or want to implement the other suggested changes feel free to do so, since I'm fairly busy for the time being.

goodtrailer avatar Jan 08 '21 02:01 goodtrailer

Hey, I had a question. Is there a way to use this as i could really use this in my streams the few upcoming weeks

daanbom avatar Feb 09 '21 15:02 daanbom

I am just wondering if it's possible to just automate the process of setting "the timer start at" the current time? Because that's how I hibernate my run manually. It would be very convenient if it's all in one button so that I don't have to type all the numbers again.

shane1595042264 avatar Mar 26 '21 16:03 shane1595042264

I am just wondering if it's possible to just automate the process of setting "the timer start at" the current time? Because that's how I hibernate my run manually. It would be very convenient if it's all in one button so that I don't have to type all the numbers again.

yeah also this doesnt work with splits

daanbom avatar Mar 26 '21 16:03 daanbom

What a pity.

From: daanbom @.> Sent: Friday, March 26, 2021 12:24 PM To: LiveSplit/LiveSplit @.> Cc: shane1595042264 @.>; Comment @.> Subject: Re: [LiveSplit/LiveSplit] Issue #952 - Added hibernate and resume run (#1774)

I am just wondering if it's possible to just automate the process of setting "the timer start at" the current time? Because that's how I hibernate my run manually. It would be very convenient if it's all in one button so that I don't have to type all the numbers again.

yeah also this doesnt work with splits

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/LiveSplit/LiveSplit/pull/1774#issuecomment-808347274 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AOW2F6IWLLKAB5D2YGWI3ALTFSYIXANCNFSM4MMDRP7Q . https://github.com/notifications/beacon/AOW2F6MDPHSQTGJSJT66O7DTFSYIXA5CNFSM4MMDRP72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGAXGNCQ.gif

shane1595042264 avatar Mar 26 '21 16:03 shane1595042264