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

On screen statistics: Issues with vertex count and suggested solutions #68

Open
Pilvinen opened this issue Jan 10, 2020 · 2 comments
Open
Labels
enhancement New feature or request

Comments

@Pilvinen
Copy link
Member

Pilvinen commented Jan 10, 2020

We are currently showing in on screen statistics a vertex count for in-view vertices.

It seems to be working fine and AFAIK the number is pulled straight from Godot (?) so it should be efficient.

However there are so many vertices that the numbers turn into exponents and the numbers lose their meaning.

Displaying either Polygons, Triangles or both would make a lot more sense to me because the numbers would be more inherently smaller. They would also be easier to compare with GPU performance specs.

Personally I would opt for both.

Implementation needs to be fast. There cannot be any overhead.

Include both:

  • In-view
  • Loaded world.
@Pilvinen Pilvinen added enhancement New feature or request good first issue Good for newcomers labels Jan 10, 2020
@Pawlost Pawlost added this to the 3 Generation optimization milestone Apr 30, 2020
@Pawlost
Copy link
Contributor

Pawlost commented Apr 30, 2020

Yes we are taking values from godot. But vertice count is only value which can be taken. This means it would require our own implementation.

@Pawlost Pawlost removed the good first issue Good for newcomers label Jun 5, 2020
@Pawlost
Copy link
Contributor

Pawlost commented Jun 5, 2020

Showing something in view is extremely hard right now so I am removing good first issue tag.

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

2 participants