search

FAForever / nomads

This project is about bringing Nomads back as playable faction via featured mod and make a serious attempt to integrate them into the main game.
12 stars 26 forks source link

[Bug]: Nomad Atlas carrier is bugged #683

Open basedbased-alt opened 1 month ago

basedbased-alt commented 1 month ago

What happened?

The Atlas is very buggy, its speed seems to be dictated by how many units it has on it. It starts quick but then as you add unit's it begins to slow down until it moves at a snail's pace but once you add enough units to the thing it gains extreme speed and can no longer drop the units on the ground. Additionally while in this fast state it also will disappear from the game if given a ferry order or if you queue up move orders using shift. I have no idea what could be causing this since I just play the game I'm not a developer so I have no suggestions on how to fix it or where the bug is located. https://streamable.com/1042qj (video of bug)

Steps to reproduce

  1. Spawn Atlas
  2. Add units to it

Expected behavior

Changes in it's speed.

AgPh0enix commented 2 days ago

I did some additional testing on this one: the max airspeed is decreased based on amount of units in the transport, then at about 2/3 capacity the problems start. Increasing "bp.Air.MaxAirspeed" to at least 11 (using a mod) resolves the issues described above, so I can confirm the problem is related to this parameter. My best guess is that the load is reducing airspeed to zero, then overflowing the variable to an extremely high value.

Edit: When "bp.Air.MaxAirspeed" is set to 11 a fully loaded Atlas transport can barely move, so its full capacity airspeed appears to be about 10 less than the configured value. Other transports have airspeeds ranging from 13.5 to 17.5 (compared to 8 for the Atlas), likely to account for this type of speed reduction. Since it will be reduced when loaded, I think increasing the base airspeed of the Atlas could be a reasonable way to solve the problem.

Garanas commented 1 day ago

This is an interesting edge case of https://github.com/FAForever/fa/pull/6309. I think the solution should made in the FA repository. We should have a release in roughly two weeks from now.