PeterTh / gedosato

The Generic DownSampling Tool
GNU General Public License v3.0
462 stars 166 forks source link

Tales of Zestiria framerate/simulation speed patch incoming #325

Closed drdaxxy closed 8 years ago

drdaxxy commented 8 years ago

Hi,

I've found a way to change Zestiria's simulation speed to accomodate for different target framerates than 30 FPS. I haven't done enough reversing to find the proper constants to change yet (right now, I need to patch several instructions/addresses to ensure everything important runs at the right speed, and I may well have missed some subsystems), and as such, while the fix is not yet stable or well-understood, the proof of concept is there.

The reason I'm writing this issue is that I'd like to ask whether GeDoSaTo would be an appropriate place to include this mod once finished, and if so, whether you have any facilities for generic code/memory patching - in the existing game plugins, I only see D3D hooking.

Best regards Niklas

EpicBurrito commented 8 years ago

I believe FPS unlockers/etc would be better off within their own project, similar to DSFix.

drdaxxy commented 8 years ago

Oh, and just in case, here's video proof: https://www.youtube.com/watch?v=S_63xeO-8KI

drdaxxy commented 8 years ago

Closing this as this mod is being incorporated into TZFix, where it's better off anyway (especially given TZFix's other patches are essential to actually maintaining a stable 60 FPS).

PeterTh commented 8 years ago

Well done! I agree it's better off there, I just hope that it's compatible with GeDoSaTo, since I like taking hudless screenshots with a controller binding.

drdaxxy commented 8 years ago

Thanks. Worries about compatibility between the two mods were why I initially figured it might be useful to have the patch included in both, but I'm happy to say there's no such problems. Playing the game with TZFix and GeDoSaTo myself, works perfectly.

Nice job on that, by the way :+1:

PeterTh commented 8 years ago

Sadly, the game speed modification isn't working for me yet, GeDoSaTo or no GeDoSaTo :/ But I made a separate issue for that in the TZF project.