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

Status of the project "ECS Bench Suite" #130

Closed
64kramsystem opened this issue Nov 19, 2022 · 5 comments · Fixed by rust-gamedev/ecs_bench_suite#33
Closed

Status of the project "ECS Bench Suite" #130

64kramsystem opened this issue Nov 19, 2022 · 5 comments · Fixed by rust-gamedev/ecs_bench_suite#33

Comments

@64kramsystem
Copy link

Somebody asked on Discord the status of the project ECS Bench Suite.

I've had a look and it seemed abandoned-ish. Is there any intention to maintain it? @ozkriff mentioned Ralith and @erlend-sh.

Generally speaking, archiving a repository makes the status unambigious. In this particular case, this is even a more sensitive topic, since presenting outdated results can be misleading.

In theory, maintaining it should be very lightweight, as API changes should be small (I haven't checked though), and the diagrams are automatically updated (after the test suite is run).

On the other hand, "serious" maintenance is on another level, as it requires discussing and critically reviewing the benchmark themselves (in practice, dealing with issues like this).

I'm no fan of microbenchmarks, so I'd archive it.

@64kramsystem
Copy link
Author

64kramsystem commented Nov 19, 2022

Reasons why I'm posting it here:

  1. that repository seems unmaintained, so this issue may not gather attention
  2. the issue Tracking: Important Issues Filed Elsewhere seems unmaintained as well

@erlend-sh
Copy link
Member

It seems to have served its purpose.

[ECS Bench Suite] arguably went quite well even though it’s not that actively updated; I like to think we collectively realized that speed is only one aspect of an ECS, and a rather small one at that once a baseline of performance has been established.

I’m in agreement with archiving it.

@alice-i-cecile
Copy link

From the Bevy perspective: I'm on board with archiving this. Definitely interested in sharing knowledge (including places where projects are leaving performance on the floor) between ECS projects going forward, but I don't think that the payoff on maintaining microbenchmarks is there.

@Ralith
Copy link

Ralith commented Nov 19, 2022

It'd have some value if it was actively maintained, but nobody's stepping up to do so, so let's drop it rather than leaving it in limbo indefinitely.

@64kramsystem
Copy link
Author

Thanks for the input 😄

I'm going to wait until Sat Nov 26, then, if there aren't any dissenting opinions, I'm going to add a Status section to the header, with the text Erlend provided, and archive the repository.

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

Successfully merging a pull request may close this issue.

4 participants