Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draw time increase after #22579 #22739

Closed
Dyrewulfe opened this issue Jan 12, 2018 · 2 comments
Closed

Draw time increase after #22579 #22739

Dyrewulfe opened this issue Jan 12, 2018 · 2 comments
Labels
<Bug> This needs to be fixed Code: Performance Performance boosting code (CPU, memory, etc.) (S1 - Need confirmation) Report waiting on confirmation of reproducibility

Comments

@Dyrewulfe
Copy link
Contributor

Game version: Any after build 7002

Operating system: Windows, Linux, Android

Tiles or curses: Tiles

Mods active: None

Draw time spiked after PR #22579. The effect is mitigated by the performance boost this provided to hardware rendering. It's extremely noticeable in software

Profiling indicates that game::draw_ter(bool) is taking roughly 12 times as long as it previously did.

@Leland Leland changed the title Regression in #22579 Draw time increase after #22579 Jan 14, 2018
@Leland Leland added <Bug> This needs to be fixed Code: Performance Performance boosting code (CPU, memory, etc.) (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Jan 14, 2018
@Leland
Copy link
Contributor

Leland commented Jan 22, 2018

Pinging @BevapDin

Also, @Graywolfe813 left some notes on the closed #22579, copying here for visibility:

I suspect there's a glitch in here somewhere. Relative hardware performance has improved compared to software(due to not having to constantly bind/unbind textures), but the overall drawing performance has decreased dramatically.

I'm running an Intel integrated, but it wasn't affected by #21317. The issue affects both Windows(OpenGL and Direct3D backends) and Linux builds, as well as my Android devices I've tested on.

Is there a logic error here causing unnecessary drawing, perhaps?

Edit: I'm asking because if there is, I'm not seeing it.

Also, the Android port UI elements are broken by this PR, which should not be the case as they're handled directly. My gut says the screen is being redrawn excessively.

@RedPine91
Copy link

RedPine91 commented Jan 26, 2018

Is this the culprit that is making craft times excessively long? Normally, when I install a vehicle part, the process is nearly instant, but now I suffer 3 sec of IRL time for every 5 min game time each installation takes. This has made interacting with vehicles nigh unbearable. EDIT: To clarify, this means a 15 minute seat installation takes IRL 8 seconds.

I parked the vehicle far from zombies, no hordes are on minimap, I have only 1 NPC, and there are only 6 cargo spaces full of loot, so this is the only thing I can think of that could be slowing the game down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Code: Performance Performance boosting code (CPU, memory, etc.) (S1 - Need confirmation) Report waiting on confirmation of reproducibility
Projects
None yet
Development

No branches or pull requests

3 participants