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 metrics and regression tracking #1039

Closed
TzviPM opened this issue Dec 14, 2018 · 7 comments
Closed

Performance metrics and regression tracking #1039

TzviPM opened this issue Dec 14, 2018 · 7 comments

Comments

@TzviPM
Copy link
Contributor

TzviPM commented Dec 14, 2018

From the October 2018 Roadmap in #937:

At some point we will want to start more aggressively measuring and tracking performance. This is a vague, open-ended problem; but ideally we would have something like a CI system for performance, that would let us run some set of real-world benchmarks against new commits, and allow us to visualize changes over time. A good first step for this would be to create a new xi-bench crate that starts putting together these sorts of real-world benchmarks.

@TzviPM
Copy link
Contributor Author

TzviPM commented Dec 14, 2018

From the README:

Incredibly high performance. All editing operations should commit and paint in under 16ms. The editor should never make you wait for anything.

@jansol
Copy link
Collaborator

jansol commented Dec 14, 2018

What are you trying to say?

@TzviPM
Copy link
Contributor Author

TzviPM commented Dec 14, 2018

Sorry, @jansol, I am trying to say that I'd like to take a stab at some performance benchmarks. Unfortunately, there wasn't any issue for that, so I made one.

@jansol
Copy link
Collaborator

jansol commented Dec 14, 2018

This issue is a bit too vague to be useful so I'm inclined to just close it. As a rule of thumb I'd say if it a) needs planning b) is somehow controversial or c) is complex enough to have a TODO list, then it needs an issue.

Performance work is definitely a massive enough project to warrant a TODO list but the current wording in the roadmap and readme is too vague to actually write one. If you have a more specific plan please open an issue for that. If you want to just add some metrics or benchmarks here and there feel free to just open a PR without creating an issue; I think most of the time those will be fairly trivial.

@TzviPM
Copy link
Contributor Author

TzviPM commented Dec 14, 2018

@jansol, I was thinking of implementing a thin wrapper around the xi-core-lib that spawns xi-editor in a thread and measures performance of commands in an e2e-style test suite. Does that make sense to you for an approach? I suppose we should just chat about this in zulip?

@jansol
Copy link
Collaborator

jansol commented Dec 14, 2018

Yes.

@jansol jansol closed this as completed Dec 14, 2018
@cmyr
Copy link
Member

cmyr commented Dec 15, 2018

@TzviPM I think this is a great project, and am happy to chat about it!

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