• Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Login
FAForever Forums
  • Categories
  • Recent
  • Tags
  • Popular
  • Users
  • Groups
  • Login

Increase the lift factor of all transports

Scheduled Pinned Locked Moved Balance Discussion
22 Posts 11 Posters 2.4k Views
Loading More Posts
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • T
    Tomma
    last edited by Tomma 10 Oct 2023, 04:12 10 Oct 2023, 04:12

    https://github.com/FAForever/fa/commit/0912e85ff31b50d286207c9ec4fa964d0ce12704
    http://patchnotes.faforever.com/balance/3761.html#transports
    The problem is after latest changes to transports they behave badly around mountains, you can see it on many ladder maps such as PoR or vulcan reach. Transport "bumps" into mountain and instantly stops, then starts sloooowly climbing up, after it has passed the mountain, it starts to descend comically slowly.
    My problem is:

    1. It looks bad.
    2. there is no indication of how big the mountains are. Would they bump, how long would they take to climb up/down?
    3. you cant even reliably fly around mountains because pathfinding might just say "fuck you" and make transport graze the side of a mountain making it slow down considerably.
    4. it happens on already existing maps which are currently in rotation, so you cant really say its "map issue".

    Skill issue

    1 Reply Last reply Reply Quote 4
    • J
      Jip
      last edited by 10 Oct 2023, 05:42

      The problem is not with the transports, but with the maps:

      d7b39f16-0d42-4913-ad22-374eb06549c9-image.png

      26df1866-6948-4972-9faf-cd306a46552d-image.png

      In particular Point of Reach; the game is not made for such tall and steep mountains. Compare these maps with 99% of the maps made by GPG and you'll see that they barely, if ever, contain these large and steep mountains.

      A work of art is never finished, merely abandoned

      1 Reply Last reply Reply Quote 0
      • T
        Tomma
        last edited by Tomma 10 Oct 2023, 05:56 10 Oct 2023, 05:54

        Yes but my point is that such maps already exist and are played. If fixing transports doesnt affect the balance, then whats the issue? Its that, or suffering on them (and all other maps with steep hills, actually i'm not sure how big the hill should be to affect transports), or removing such maps from map pool.

        Skill issue

        1 Reply Last reply Reply Quote 0
        • J
          Jip
          last edited by Jip 10 Oct 2023, 06:02 10 Oct 2023, 06:00

          We adjusted the statistics of the transports to make picking up and dropping off units more reliable (on the average map). Units inherit the velocity of the transport as they are detached when dropping them off. If the transport lands too fast (lift factor) then the units can fly off quite a distance (1 to 3 land factories far). If they end up in a mountain, a large field rock, a structure or some other unpathable bit of terrain then they are destroyed upon contact with the terrain. That is why the lift factor is quite a bit lower than before - it almost guarantees that the transport is only moving vertically upon dropping.

          That's the problem 🙂 , as always: I'm open to a (different) reliable fix to both (reliable landing and quicker lift off) problems. But I'm not going to spent time on it when the current solution only impacts maps that look like this:

          2885ee2a-6073-44e7-b5c8-23af0e635a1f-image.png

          A work of art is never finished, merely abandoned

          1 Reply Last reply Reply Quote 2
          • C
            Ctrl-K
            last edited by 10 Oct 2023, 06:11

            Also noticed that issue even on map gen it happens. Is there an acceleration for this one?

            “Be a yardstick of quality. Some people aren’t used to an environment where excellence is expected.”
            — Steve Jobs.
            My UI Mods
            Support me

            1 Reply Last reply Reply Quote 1
            • T
              Tomma
              last edited by 10 Oct 2023, 07:54

              Also i noticed that t3 uef transport has lift factor of 4, which is already much more human, and apparently it has no issues offloading units.

              Skill issue

              1 Reply Last reply Reply Quote 0
              • J
                Jip
                last edited by 10 Oct 2023, 09:22

                @Tomma isolating properties won't work towards a solution; the continental is at a higher average altitude and therefore has more time to stabilize before dropping units.

                A work of art is never finished, merely abandoned

                1 Reply Last reply Reply Quote 0
                • S
                  snoog
                  last edited by 10 Oct 2023, 11:50

                  I remember a game maybe a month or two ago that my ACU was thrown maybe 3-4 factories in distance as the transport was killed. Somehow survived too lol

                  1 Reply Last reply Reply Quote 0
                  • C
                    clyf
                    last edited by 10 Oct 2023, 12:43

                    I've been experimenting with using blueprint buffs to modify the flight characteristics of transports based on whether they're dropping off/picking up/flying etc. If there's a reliable fix to be made, I think it's there.

                    (Hadn't occurred to me before, but the same technique could also address the leisurely pace with which aircraft enter carriers)

                    J 1 Reply Last reply 10 Oct 2023, 14:02 Reply Quote 2
                    • J
                      Jip @clyf
                      last edited by 10 Oct 2023, 14:02

                      @clyf said in Increase the lift factor of all transports:

                      I've been experimenting with using blueprint buffs to modify the flight characteristics of transports based on whether they're dropping off/picking up/flying etc. If there's a reliable fix to be made, I think it's there.

                      How would this work? I'm not aware of a way to edit the values on a unit-to-unit basis

                      A work of art is never finished, merely abandoned

                      C 1 Reply Last reply 10 Oct 2023, 14:49 Reply Quote 1
                      • Z
                        ZLO
                        last edited by 10 Oct 2023, 14:26

                        https://www.twitch.tv/videos/1945822700?t=05h58m22s
                        this was one of the fun moments from the summer tourney

                        TA4Life: "At the very least we are not slaves to the UI" | http://www.youtube.com/user/dimatularus | http://www.twitch.tv/zlo_rd

                        1 Reply Last reply Reply Quote 2
                        • C
                          clyf @Jip
                          last edited by 10 Oct 2023, 14:49

                          @jip

                          Just from looking at the BuffBlueprint setup (again, not something I've really exhaustively nailed down) it looks like it's possible to set up a couple of different flight profiles ("cruise", "takeoff", "air brake") and apply them on a per-unit basis, like adjacency?

                          1 Reply Last reply Reply Quote 1
                          • W
                            waffelzNoob
                            last edited by 6 Nov 2023, 15:44

                            Just curious if this is being worked on?

                            frick snoops!

                            C 1 Reply Last reply 6 Nov 2023, 17:21 Reply Quote 0
                            • C
                              ComradeStryker @waffelzNoob
                              last edited by 6 Nov 2023, 17:21

                              Transports going over flat terrain: 👍
                              Transports going over a tiny difference in height: 😵

                              They're slower when changing altitude than my thought process...

                              ( ͡° ͜ʖ ͡°)

                              1 Reply Last reply Reply Quote 0
                              • phongP
                                phong
                                last edited by phong 11 Jul 2023, 05:52 7 Nov 2023, 05:52

                                Happened on mapgen for me as well recently. I hope your experiments result in a successful fix clyf, thanks for looking into this.

                                1 Reply Last reply Reply Quote 0
                                • C
                                  clyf
                                  last edited by 8 Nov 2023, 14:07

                                  This post is deleted!
                                  1 Reply Last reply Reply Quote 0
                                  • C
                                    clyf
                                    last edited by clyf 11 Aug 2023, 14:29 8 Nov 2023, 14:19

                                    It's on my list, and very near and dear to my heart. Some very general updates:

                                    • I've become intimately familiar with the OnMotionEventChange functions during some recent work on the Fire Beetle, so I'm confident the front piece--reliably and tidily detecting (read as: without a forked thread) when the transport is landing/picking up/taking up--is possible.
                                    • The end bit (a working flight profile) has always been possible.
                                    • That leaves the middle bit, actually modifying the transports themselves on an individual basis.

                                    Just for kicks, the behavior I'm trying to emulate:
                                    alt text
                                    (Ignore the engine failure part, engine failure and descending as fast as possible have a lot in common!)

                                    1 Reply Last reply Reply Quote 7
                                    • C
                                      clyf
                                      last edited by 8 Nov 2023, 20:37

                                      ziggity:
                                      https://github.com/FAForever/fa/pull/5653

                                      None of that BuffBlueprint bs (that probably won't work, as Jip surmised), but we can bypass the problem entirely by simply telling the transports to unload when they're very close (1-2 ogrids), as opposed to the extremely close currently asked for by the engine (<1 ogrid).

                                      With a little flexibility ("little" as in, a rounding error compared to that introduced by a group unload order) in the dropoff point, we're free to adjust the flight performance parameters to whatever we like. See the Courier in the above PR.

                                      1 Reply Last reply Reply Quote 2
                                      • J
                                        Jip
                                        last edited by 10 Nov 2023, 11:03

                                        These changes are now live on the deploy/fafdevelop branch. We'll ship it with the next and hopefully last hotfix. Please give it a try and report back here.

                                        With thanks to @clyf for his initiative and effort in this 🙂

                                        A work of art is never finished, merely abandoned

                                        1 Reply Last reply Reply Quote 1
                                        • AmygdalaA
                                          Amygdala
                                          last edited by Amygdala 11 Oct 2023, 12:34 10 Nov 2023, 11:16

                                          Just tried it and it feels like a huge improvement. Transports are way more responsive, drop quicker and are much faster when traversing mountains. great job!

                                          1 Reply Last reply Reply Quote 0
                                          • First post
                                            Last post