JACoders / OpenJK

Community effort to maintain and improve Jedi Academy (SP & MP) + Jedi Outcast (SP only) released by Raven Software
GNU General Public License v2.0
2.01k stars 612 forks source link

Featurerequest - Overwrite #define values for hyperspace with values from entity trigger_hyperspace #1047

Open MoritzJT opened 4 years ago

MoritzJT commented 4 years ago

As the title suggests - this would be a welcome addition if those defines here: could be overwritten per map by the values of the trigger hyperspace entity and use trigger specific values instead so that multiple trigger_hyperspace entities can coexist with different distances to work with. Right now the time to travel is hardcoded at 4 seconds making quick jumps impossible with fast ships.

ensiform commented 4 years ago

because the defines are there means it's shared between cgame & server game and thus moving it is problematic to prediction and network/save

Razish commented 11 months ago

+1 for closing as out of scope. This would require both server and client changes for a good experience and so is better suited for a mod.

Is it worth experimenting with only server-side changes to see how poor the behaviour really is for clients without the updates? Is there a use-case for instant hyperspace which doesn't rely on good prediction? I'm seeing some +/- 500 and 1000 around this code. I'm not familiar, but does that mean there are 3 stages (enter, travel, exit) and we could reduce that to just enter + exit without client modification?

If that's not feasible or desirable, I'm in favour of closing.