cstar_perf icon indicating copy to clipboard operation
cstar_perf copied to clipboard

GUI should help make sure comparisons are against the same baseline

Open belliottsmith opened this issue 10 years ago • 5 comments

Right now (especially close to release) we're having problems with patches with different baselines being compared to each other. It would be great if cstar presented the most recent SHA commits for you to select from for a given branch, so that when the test is actually run we know for certain they were compared like-for-like.

belliottsmith avatar Sep 16 '15 14:09 belliottsmith

I could probably do something similar to what I've done with stress_revision for this. At startup, I replace the revision name by their SHA.

aboudreault avatar Sep 16 '15 14:09 aboudreault

It would be helpful to show it to the user, though, so that they actually check the branches haven't diverged too much. Perhaps just showing the SHA you will pick, along with the last 5 commit messages would be enough - if they're out of whack, they can fiddle with the SHAs / rebase themselves.

belliottsmith avatar Sep 16 '15 15:09 belliottsmith

I would recommend we do this entirely client side via the github API, that way we can stay away from needing to check out each branch on the frontend.

EnigmaCurry avatar Sep 16 '15 16:09 EnigmaCurry

I've been thinking about how to refactor how we specify revisions as a whole. Please see #98.

I think the changes here would follow nicely on that.

For example, you could start typing in the revision box : "belliottsmith/". As soon as you type that '/' the browser could do an ajax call to this API : https://developer.github.com/v3/repos/commits/ and display the recent commits in a drop down box for that branch.

EnigmaCurry avatar Sep 16 '15 17:09 EnigmaCurry

That sounds neat, yeah. If it could scope it by branch and then, within that branch, the individual recent commits (with their messages) it would be really awesome.

On 16 September 2015 at 18:10, Ryan McGuire [email protected] wrote:

I've been thinking about how to refactor how we specify revisions as a whole. Please see #98 https://github.com/datastax/cstar_perf/issues/98.

I think the changes here would follow nicely on that.

For example, you could start typing in the revision box : "belliotsmith/". As soon as you type that '/' the browser could do an ajax call to this API : https://developer.github.com/v3/repos/commits/ and display the recent commits in a drop down box for that branch.

— Reply to this email directly or view it on GitHub https://github.com/datastax/cstar_perf/issues/97#issuecomment-140806824.

belliottsmith avatar Sep 16 '15 17:09 belliottsmith