stohrendorf / CroftEngine

An open-source Tomb Raider 1 engine remake
Other
285 stars 18 forks source link

switch to 60 fps #183

Closed stohrendorf closed 2 years ago

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 67b9ad425589a874c09b154841d939ea8ba60be9 into 72dcd8c0ba888b56363069ae897408fd25fac2d1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging c65657791968e5ef04af5699548a37cf1ae1038f into 72dcd8c0ba888b56363069ae897408fd25fac2d1 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 68158b4c24845eb4da58a972544464fa8d1840da into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 981145b948a09299949a0b06dc58c862a82c9ff2 into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging f1d1767179df2c0cfb36c7e392521957a937eaf2 into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging 7198111bda669779e520c814ca32e8d884dbadc8 into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 3 years ago

This pull request introduces 2 alerts when merging d77ddd36c28bd5820e22e5f19a856dcb480d577e into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 579708122f9e92406dd24d3ce7aff519ee9229da into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 153a32e0e53c20e19d542450c4ca239e3362ac2d into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 5554c89af13fee4a2a6435a27330582a08085ac7 into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 1a567ff41154f9590e7b12681c806d3a153245ab into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 23b92a443f26002d02d6916643d2b5f78764b319 into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging cf2b3cbd72fda6f3d26256045629d5a92fca0c8a into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 4e179458597a3ca299b3d40163df9abd3a751974 into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging e14b63b669eb4abbdad2d1ff5b66a4c1f92f852e into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 0844c20d93460b3e9ccc224339a58b04f91f330d into f7ddcd1193b49a8c9f305c0f89ae3eea025ac6ff - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging d2ea9d8ed498a0dd5734c4f9071ee1ed32de0fa3 into 05352d4aee20185b371b7db6ec39852fccfc59b3 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging de2188a4f67a458e6152d8f87b60900f9c41c1e9 into 05352d4aee20185b371b7db6ec39852fccfc59b3 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging b5bbc8694095466c166a6d76fa157bd2a2aa4f4e into 6586915e13e97be5798ca625dbda3b7d8b3e7f97 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 91d71277cf0187bf2ca08cf5c7b9f170205eca24 into b59561d66af7a5f8dc6d1bd6cbf13a07a7e8d78f - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 14f3983dcfae784e17759ef6c4b8df60003a9492 into 6124e6d10f44ca13503fe8a44f0312ca0cf7f719 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 7f07b2b91dd3f225302d3612cea825648c10dc4c into 6124e6d10f44ca13503fe8a44f0312ca0cf7f719 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging d881db06c86bce153462fb886bbe8c1c36721dde into 6124e6d10f44ca13503fe8a44f0312ca0cf7f719 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging 946053dc1e01983edef8fac8a35b950ea44a4184 into d92f3c584d14993aec18b62c087165e290db129b - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging ef0a54097835bff5fb440600312c95adf79a1d24 into 1b81bd79c46511f84809c5c4bb37f7d74f365f00 - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging c1bb5c8c6b2939d84a7b91b1799234c906961c4b into a09f41f0600282e357fcd7c9e133c557ad72c3eb - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 1 alert when merging b56b37a9c1e0e2fa9c5b5200ea91dc03007f7ff7 into 3e574d6b55fa2dbe28b3becaf2fe806071a87eab - view on LGTM.com

new alerts:

Sol1vaN commented 2 years ago

Sorry for my ignorance, but its possible that you make even more 60FPS feature? Like the XProger code (OpenLara) that's already have FPS unlocked (even for 120FPS, or more)

Source: https://codeload.github.com/XProger/OpenLara/zip/refs/tags/latest

Right now I've 120hz monitor, and OpenLara looks sweet with vzync 120fps@120hz

stohrendorf commented 2 years ago

Unfortunately I encountered a lot of problems with the 60 fps experiment, because it breaks the original gameplay in a lot of ways which I wasn't able to solve yet. Building it for 60 fps only already helped me avoid a lot of other problems. I stopped working on this a while ago because of these problems.

Sol1vaN commented 2 years ago

Unfortunately I encountered a lot of problems with the 60 fps experiment, because it breaks the original gameplay in a lot of ways which I wasn't able to solve yet. Building it for 60 fps only already helped me avoid a lot of other problems. I stopped working on this a while ago because of these problems.

Ooh that's is very unfortunate to see this 😥 I see the same problems with the 60FPS case on others engines. This guy (Kaze Emanuar) explain the way he approach this to solve this challenge on Mario 64 engine.

You can take a look: https://www.youtube.com/watch?v=t_rzYnXEQlE

Or: You can take another engine like XProger code, but ey! I know all this take a lot of time and effort, your work until now is impressive, thanks for this!

Even with the 30FPS locked the game looks charming, again, thanks for this 😎👍

stohrendorf commented 2 years ago

OpenLara has a completely different architecture than this engine, so it won't help me figuring this out.

stohrendorf commented 2 years ago

Closing this because it's outdated beyond any hope of recovery.