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

Moving ecs_bench to GDWG #89

Closed
erlend-sh opened this issue Feb 7, 2020 · 4 comments
Closed

Moving ecs_bench to GDWG #89

erlend-sh opened this issue Feb 7, 2020 · 4 comments

Comments

@erlend-sh
Copy link
Member

erlend-sh commented Feb 7, 2020

I talked to @lschmierer of ecs_bench and he’d be happy to move the repo to our org for community maintenance.

I think there are some open feature request about other/new ecs libraries. As I currently don't have the capacity to work on the project, I think it would benefit from moving it to the workgroup.

Some notes on the workflow: As far as I remember, I pushed all changes and updates to the bench branch. The benchmark was then run automatically by Travis and the results were pushed back by Travis to the master branch. I think that needs to be documented somewhere.
Might also make sense to migrate to the new GitHub Actions, but haven't really looked into it.

This seems like the kind of low-maintenance library that this WG is well suited for. Benchmarks benefit from neutrality, and the individual library authors have an incentive to help maintain this crate so the burden isn’t really on the wg to keep it up to date.

@Ralith
Copy link

Ralith commented Feb 7, 2020

Worth noting that running benchmarks on shared machines (i.e. any free CI provider) is a bit dubious, especially if granted the appearance of being an official judgement.

@erlend-sh
Copy link
Member Author

@TomGillen has created a new, more advanced ECS benchmark suite: https://github.com/TomGillen/ecs_bench_suite

He is "not adverse to moving it".

@AlexEne
Copy link
Member

AlexEne commented Aug 21, 2020

@erlend-sh ping me if you need any extra permissions to do the move?

@erlend-sh
Copy link
Member Author

This is now done! Further coordination measures will be discussed here: rust-gamedev/ecs_bench_suite#3

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

No branches or pull requests

3 participants