|
Notes |
|
|
|
|
Can you make a demo illustrating the broken behavior? |
|
|
|
(0007771)
|
|
Edward-san
|
2013-12-30 21:52
(edited on: 2013-12-30 21:53) |
|
Attached two simple jump demos with DOOM2.WAD in map01 which use zandronum 1.2 and zandronum 2.0beta.
|
|
|
|
|
|
Single player demos are not supported in Zandronum and thus usually go out of sync horribly. Did you check that when playing demos the behavior is consistent with what you originally observed when recording them? |
|
|
|
|
|
|
|
(0007789)
|
|
Konar6
|
|
2013-12-31 12:11
|
|
|
If someone tells me what should I look for on which map (how to reproduce it as quickly as possible), I can get the revision where the problem was introduced. |
|
|
|
|
I tested a demo that got out of sync on zandronum 2.0, the weird part is that the first try that I did to run that I got a fatal error, the second try the first map was ok, but the second was doing like parkour + moonwalk at a jumpmaze map lol...
Does the demos need a tracker?
Also I'm going to test that build to check if the demos still crash |
|
|
|
(0007794)
|
|
Edward-san
|
2013-12-31 14:03
(edited on: 2013-12-31 14:07) |
|
Quote
Single player demos are not supported in Zandronum and thus usually go out of sync horribly. Did you check that when playing demos the behavior is consistent with what you originally observed when recording them?
I checked the demos before uploading and they were working properly when I used 'zandronum -playdemo jump_1.2.lmp' and 'zandronum -playdemo jump_2.0beta.lmp'...
[edit]I tried to record a demo with multiplayer 2.0 beta, but it seems completely broken with constant jittery ... using linux x64, btw
|
|
|
|
|
Konar6 and everyone else this is really easy to reproduce. First, in Zandronum 1.2, in Map01 jump off the first ledge at the very beginning and hold jump before hitting the ground to see how it should behave: notice that there is a delay before you jump again.
Now, in 2.0 do the same (jump off and hold jump before hitting the ground), now notice that you jump or "bounce" off the ground immediately on hitting the ground, rather than there being a delay. |
|
|
|
|
Quote
In addition to the demos, can somebody go through these unofficial 2.0 beta builds and check in which build the problem first appeared?
Zandrodev2.0-131026-1630 is where the bug was introduced. |
|
|
|
(0007800)
|
|
Konar6
|
|
2013-12-31 16:20
|
|
|
|
|
|
|
Does this happen in gzdoom 880? |
|
|
|
|
"- P_ZMovement() temporarily disables jumping after a landing. Don't do this if the jump timer is already running or for short falls (e.g. along the edges of slopes, since the slope floorz calculation is pretty crappy.)"
This is likely the culprit. |
|
|
|
|
Quote
Does this happen in gzdoom 880?
Doesn't happen in GZDoom 1.8.2 if that's what you mean. |
|
|
|
|
|
|
|
|
Quote
No, I mean with this build.
Yes, it happens on this build. |
|
|
|
|
|
Then it is a (G)ZDoom bug. Can somebody check when this was fixed? |
|
|
|
|
|
|
|
|
|
Yeah, that's quite possible. ZDoom revision 2238 broke this, 2970+2979 may fix this. I'll have a look. |
|
|
|
|
|
Incidentally, I am concerned these updated jump physics could still cause possible compatibility issues even when this bug is fixed. Could I request a compat flag, something like comat_oldzdoomjumptimer? |
|
|
|
|
|
Let's not talk about new compat flags unless we know that we really need them. |
|
|
|
(0007861)
|
|
Torr Samaho
|
2014-01-03 11:05
(edited on: 2014-01-03 11:06) |
|
Zandronum's and ZDoom's jump code differ more than I thought. Please check if this fixes the issue. Also please check if the high jump rune and jump pads work as they did in 1.2.
|
|
|
|
(0007867)
|
|
Hypnotoad
|
2014-01-03 15:13
(edited on: 2014-01-03 15:23) |
|
Okay, I just wrote 5 paragraphs of feedback on this, only to be met with the error "Invalid form security token. This could be caused by a session timeout, or accidentally submitting the form twice.", going back caused my message to be COMPLETELY deleted, this is very VERY annoying.
Suffice to say, the old issue is fixed but there are now other major issues which will break many jumpmaze maps: spring pad zones (skulltag thing) are broken and don't bounce you properly anymore, actor/thing bunnyhopping is totally different - breaking maps, and jump timer behavior is very different, the latter issue may not be a bug but will definitely break many maps from all of the jumpmaze wads (as I found in the testing, e.g. 2 maps were rendered impossible and others were much harder), and possibly affect other wads (e.g. CTF relies on platforming for successful flag runs in many maps, this could affect that). My post that got deleted went into detail about all of these issues after testing over 32 maps in jumpmaze 2, I haven't the heart to recall all of the information I found again but I noted at the end that it would be very complicated to fix each of these issues individually while maintaining the new physics, which is why I strongly recommended again the compat_oldzdoomjumptimer to restore the old jumping code.
edit: forgot about the second part of your post. High jump runes do not seem to be affected. Jump pads - well there are many different ways people create jump pads, however if you mean using spring pad zones, then yes these are broken, holding jump before landing on them does not bounce you anywhere near as high as they used to on the first landing but instead simply bounces you back to where you fell from, this will break various maps.
|
|
|
|
|
Quote from Jroc
Okay, I just wrote 5 paragraphs of feedback on this, only to be met with the error "Invalid form security token. This could be caused by a session timeout, or accidentally submitting the form twice.", going back caused my message to be COMPLETELY deleted, this is very VERY annoying.
This is very annoying indeed. Happens to me often, so I just always copy a message to the clipboard before submitting. I think it's a Mantis security feature. Blzut3 should be able to tell you more about this.
Regarding the jumping: AFAIK ZDoom doesn't intend to break existing maps, but nevertheless often does so unintentionally. Instead of adding yet another compat flag, IMHO it would be better to fix this in ZDoom. Are there any ZDoom compatible maps that work as intended in Zandronum 1.2 / ZDoom 2.4.1 and break in Zandronum 2.0 / ZDoom 2.5.0? |
|
|
|
|
At least with jumpmaze maps, the problem is jumpmaze flat out doesn't work in ZDoom (2.4.1 or 2.5.0) so you can't test properly; it has different air movement: jumpmaze relies on compat_limited_airmovement being 0 in zandro, as far as I know there is no way to replicate standard zandronum air movement in Zdoom.
I doubt these changes had any noticeable affect on zdoom compatible maps, jumpmaze is pretty much the only project I know of that relies on very specific jumping code, with the possibility of CTF also (have not tested yet), and jumpmaze is exclusive to ST/Zandro. This is why I think it should be a compat flag for zandro since changing the code again for zdoom probably wont have much use for any zdoom projects. |
|
|
|
|
|
Let's try something else. Since the jumping behavior introduced in ZDoom revision 2238 is apparently broken, I'll revert this change instead of pulling even more ZDoom changes. Please test if this binary shows the 1.2 jumping behavior again. When we further upgrade our ZDoom base to also include the other jumping changes, we can still introduce a compat flag. |
|
|
|
|
This appears to have restored everything back to normal, thanks.
I think the jumping physics and air movement we have now should be locked in as "skulltag physics", since this is how it has been in skulltag for as long as I can remember (several years, possibly since 2005). So you could have a flag like compat_skulltagphysics which instantly restores the physics to this state, in case there are any changes in the future. |
|
jump_1.2.lmp (2,332) 2013-12-30 21:51