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

Add support for MultiMesh GroundedSpatial Assets for better performance #139

Open
kb173 opened this issue Sep 18, 2019 · 3 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@kb173
Copy link
Member

kb173 commented Sep 18, 2019

There are a lot of individual assets and even more individual meshes when having multiple large PV plants active. This causes a noticeable performance drop.

We should omptimize the model (fewer individual meshes, ideally just 1 material) and maybe use a multimesh for the fields.

@kb173 kb173 added the bug Something isn't working label Sep 18, 2019
@kb173 kb173 added this to the Version 0.4 (Showcases) milestone Sep 18, 2019
@kb173 kb173 self-assigned this Sep 18, 2019
@kb173
Copy link
Member Author

kb173 commented Sep 18, 2019

6f653ed made a big improvement, but the model could still be simplified: e.g. individual 3D panels aren't necessary, we could have one big flat mesh with a repeating texture (and perhaps switch to the more detailed model when the player is close?)

@kb173
Copy link
Member Author

kb173 commented Sep 21, 2019

Around 5 nearby large PV fields seem to work well performance-wise. I think the bottleneck is actually the high amount of Nodes, not the meshes.

Let's see on Monday whether we need to investigate this more and improve performance, or whether we won't have that many PV fields on one place anyways. If we do need to improve it, turning it into a multimesh should work.

@kb173
Copy link
Member Author

kb173 commented Sep 24, 2019

We placed assets pretty densely on Monday and the performance didn't seem to be a problem - seems like reducing the nodes (6f653ed) was enough.

Generalizing this issue for maybe in the future

@kb173 kb173 added enhancement New feature or request and removed bug Something isn't working labels Sep 24, 2019
@kb173 kb173 changed the title Having several PV fields has noticeable performance impact Add support for MultiMesh GroundedSpatial Assets for better performance Sep 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant