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

      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
      • JipJ
        Jip
        last edited by

        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

          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
          • JipJ
            Jip
            last edited by Jip

            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
            • Ctrl-KC
              Ctrl-K
              last edited by

              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

                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
                • JipJ
                  Jip
                  last edited by

                  @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

                    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

                      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)

                      JipJ 1 Reply Last reply Reply Quote 2
                      • JipJ
                        Jip @clyf
                        last edited by

                        @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 Reply Quote 1
                        • ZLOZ
                          ZLO
                          last edited by

                          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

                            @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
                            • waffelzNoobW
                              waffelzNoob
                              last edited by

                              Just curious if this is being worked on?

                              frick snoops!

                              ComradeStrykerC 1 Reply Last reply Reply Quote 0
                              • ComradeStrykerC
                                ComradeStryker @waffelzNoob
                                last edited by

                                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

                                  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

                                    This post is deleted!
                                    1 Reply Last reply Reply Quote 0
                                    • C
                                      clyf
                                      last edited by clyf

                                      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

                                        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
                                        • JipJ
                                          Jip
                                          last edited by

                                          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

                                            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