GithubHelp home page GithubHelp logo

Comments (16)

Garanas avatar Garanas commented on August 24, 2024 1

@MrRowey KionX is right - this issue should not get closed yet.

I think the height is set that high to make sure it is visible on maps with large height differences. Regardless, this would be a prime candidate for an executable change. Sadly, I don't have the experience to perform that type of work.

from fa.

Crotalus avatar Crotalus commented on August 24, 2024

A better solution was to use the selection overlay checkbox to control if you want single unit select circles or not (it was just a keybinding before)

from fa.

aeoncleanse avatar aeoncleanse commented on August 24, 2024

Don't remove options entirely. Mess with defaults sure, but don't remove, some people with fast machines will probably use those options.

As for me, I have my fingers crossed this can be properly fixed at some point. My FPS in supcom doesn't dip under 60 on a 560m, until I select units with rings turned on, which instantly drops me to 10FPS. It sucks.

from fa.

aeoncleanse avatar aeoncleanse commented on August 24, 2024

You say the merge overlay circles code is crap. Do we have access to improve it?

from fa.

Crotalus avatar Crotalus commented on August 24, 2024

You say the merge overlay circles code is crap. Do we have access to improve it?

Possibly, we would need to change the range circle shader code.

from fa.

aeoncleanse avatar aeoncleanse commented on August 24, 2024

I've had my uncle take a look at this as it appears in game, not the code. He's an expert programmer who specialises in geometric functions for CAD operations, and his jaw dropped when he saw the impact these rings had on FPS. I think it's essential we find out how they are calculated, how the intersect is calculated, and why the hell this FPS drop is happening. On my machine, even selecting 3 ASFs drops my FPS from 60 to 25.

Crucial: Zoom level affects the FPS, so it does appear to be the display aspect rather than the calculation which is causing it.

from fa.

aeoncleanse avatar aeoncleanse commented on August 24, 2024

Since my last post on this, I've upgraded my laptop. The old one had a 560m GPU and an i7 CPU. The new one has an i7 which is slightly faster, but not by much. However, the GPU is a 980m, a SIGNIFICANTLY faster GPU. For me, this issue has completely vanished with the new machine, confirming this is definitely a rendering issue not a CPU side issue.

from fa.

Crotalus avatar Crotalus commented on August 24, 2024

Probably fixed by Eximius here:

3cb462d

from fa.

ChrisKitching avatar ChrisKitching commented on August 24, 2024

Definitely not fixed by Eximius there, but made very slightly less awful.

You really should lurk in Slack more, Crotalus, it's where the low-latency stuff happens.

from fa.

Sheeo avatar Sheeo commented on August 24, 2024

The bug remains, but unfortunately until we get access to the source so we can optimize the rendering through other means, it's tough.

from fa.

HUSSAR-mtrela avatar HUSSAR-mtrela commented on August 24, 2024

too bad we cannot fix it now because even with GTX 970 and i7-4930K I see fps drop a lot when selecting a lot of ASFs. Also, there might be something wrong with range rings on SCUs because selecting just a few of them causes fps drop as much as selecting a lot of ASFs... I will investigate it more.

from fa.

Sheeo avatar Sheeo commented on August 24, 2024

@HUSSAR-mtrela this cannot possibly be a high priority unless you have found means for changing the render calls.

from fa.

MrRowey avatar MrRowey commented on August 24, 2024

@Garanas I'm assuming this can be closed with the Merged PR that is Mentioned here ?

from fa.

KionX avatar KionX commented on August 24, 2024

PR #3004 is just a garbage cleaning.
The performance problem remained.
Fixing this is difficult and possible only in the engine.
It's about fillrate video card. A circle is drawn from 2 cylinders.
These cylinders are very large in height. This overloads video card.
And if you draw circles from a bunch of units, then slow down occur.
So eliminating the double rendering doesn't completely solve the problem.
It is also necessary to reduce the height of the cylinders.

The description of my research begins here(Rus):
https://faforever.zulipchat.com/#narrow/stream/203556-development-russian/topic/.28no.20topic.29/near/189103149

from fa.

4z0t avatar 4z0t commented on August 24, 2024

Yeah, it is true, they really cause fps drops, I experienced that even having top gpu, the only solution rn is turn off anti aliasing... but it only reduces the effect

from fa.

Garanas avatar Garanas commented on August 24, 2024

This is not fixed, but improved significantly by an engine patch (FAForever/FA-Binary-Patches@d860211) and two shader adjustments (a8a1c9b and 6f499ee). See also the forum post with the latest sneak peaks!

from fa.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.