TASEmulators / BizHawk

BizHawk is a multi-system emulator written in C#. BizHawk provides nice features for casual gamers such as full screen, and joypad support in addition to full rerecording and debugging tools for all system cores.
http://tasvideos.org/BizHawk.html
Other
2.14k stars 380 forks source link

[F.Rq] 'Play Movie' dialog improvements #3735

Open RetroEdit opened 1 year ago

RetroEdit commented 1 year ago

I was just talking about this in Discord, and I realized I don't really use the 'Play Movie' dialogue anymore, and I haven't used in the last few months when I've been using BizHawk fairly regularly. There are a few reasons for this (maybe some of these could be split into separate issues).

Morilli commented 1 year ago
  1. Could you give some example values of what you're seeing, like how many movies do you have, how long does the dialog load? fwiw, testing locally with 2700 backup tasproj files with about 1.5 minute length each, the dialog only takes about 1 second to load, which I find acceptable. That said however, I do agree that having to read the entire input log for the sole purpose of displaying the frame count of each movie is ridiculous.
  2. Would changing the global Movies folder path work for you?
  3. You can right-click a movie in the dialog and select Edit, which probably opens the default application you have set; for more it opens the bk2 in 7-zip which means I can just edit it.
RetroEdit commented 1 year ago
  1. It's been a while since I've tested and I've gotten a better computer since then. But to my recollection, it would sometimes take 30 seconds for loading movie files whenever I had multiple larger files in the directory. I'll try and do some tests to see if this is still a concern, but I think the performance might differ if you had a few larger files that were several hours each.
  2. No, changing the global Movies path really wouldn't. Because each movie is in a game-specific directory as I described. (I don't deny that this might just be a personal issue with my workflow being incompatible with the general workflow BizHawk was designed for)
  3. Sure, that's fine, but that's more of a workaround, and you can only edit that when the movie isn't already open in BizHawk. And that's basically what I already do, minus using Play Movie part. Edit: specifically, certain things like the author field should be freely editable, because they don't affect sync. And they are editable in TAStudio.
YoshiRulz commented 1 year ago

Make sure to test on a dev build that includes https://github.com/TASEmulators/BizHawk/commit/fe845ce8efae8fa3b11d04079d3ae20ec844ea8a

re: 2) a path system redesign would benefit you then?

RetroEdit commented 1 year ago

I'll be sure to test using a dev build that includes fe845ce. As for 2), maybe? Hm. I'm a bit sleep-deprived and don't currently have full capacity brain power to think through that right now, but that could actually benefit me in general if I normalized my own directories a bit more. I also store Lua scripts and notes near the movie file, though I'm a bit inconsistent on the exact directory structures. To be honest, I was considering consolidating Lua scripts though so they could be redistributed in a more sensible way.

RetroEdit commented 1 year ago

Okay, I re-tested the Play Movie slowness. Looks like that was fixed sometime between 2.8 and 2.9.1.

YoshiRulz commented 1 year ago

Hey look what I found :P #2198

RetroEdit commented 1 year ago

Hey look what I found :P #2198

Yep, I linked that in the original post a few days ago. 😄

Edit: Oh, it was originally titled 'The Play Movie dialog is slow'... I've been a bit bad about accidentally repeating my own issues recently, though it's usually been with different details. This one does have some different details. Oh, and I mentioned the double-loading issue that @Morilli recently fixed.