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

Performance hit when flight plan window is opened #2157

Closed
ghost opened this issue May 11, 2019 · 4 comments · Fixed by #2163
Closed

Performance hit when flight plan window is opened #2157

ghost opened this issue May 11, 2019 · 4 comments · Fixed by #2163
Labels
Milestone

Comments

@ghost
Copy link

ghost commented May 11, 2019

FPS keeps at 60 when flight plan window is closed, even with sigificantly long plan is plotted.
But when flight plan window is opened and with a long plan present, the game can be a bit laggy, running lower than 40fps, and drop lower than 30 when moving the map around.
CPU usage is around 45% and GPU is below 10% when lagging, so it is likely not caused by hardware limitation.

@ghost
Copy link
Author

ghost commented May 11, 2019

My description was a bit wrong. Only when there's a manoeuvre, a long flight plan and flight plan window is opened there will be FPS drop

@pleroy pleroy added the bug label May 11, 2019
@pleroy
Copy link
Member

pleroy commented May 11, 2019

I was able to reproduce the problem.

@pleroy
Copy link
Member

pleroy commented May 11, 2019

Profiling a journal:

Read                         23%
FlightPlanReplaceLast        17%
UpdatePrediction             12%
PlanetariumPlotPrediction    6%
PlanetariumPlotFlightSegment 4%

@eggrobin
Copy link
Member

For future reference: this issue was introduced in Fano, not Fáry (that is, during the restructuring of the UI code, rather than the introduction of text input).

Sorry, something went wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants