These are chat archives for PerfDotNet/BenchmarkDotNet

18th
Jan 2016
Matt Warren
@mattwarren
Jan 18 2016 12:10
@AndreyAkinshin I just pushed one extra change (PerfDotNet/BenchmarkDotNet@9e16e8c) to make the Delta Result Extender a bit more robust. If that could go into 0.8.2 as well that would be cool.
BTW I think that the ChangeLog is a really nice idea. It also shows that BenchmarkDotNet is becoming a more mature OSS project, good job!
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 12:46
@mattwarren, Ok, no problem. I will test the last develop version tonight and publish it.
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 17:12
@mattwarren, your last merge contains errors, I can't build our solution in the develop branch.
You have removed my BenchmarkStatResultExtenderTests and spoil BaselineDeltaResultExtenderTest, PerformanceUnitTest, see diff with parent 140deccc.
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 17:25
I made a rollback. Please, review it.
Matt Warren
@mattwarren
Jan 18 2016 20:11
Uggh, I'm sorry about that. It's weird I had a problem with the merge, but I thought that I double-checked it by downloading the latest code and building it from scratch
Did you manage to fix it, or do you need me to do something?
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 20:12
I already fixed it.
Matt Warren
@mattwarren
Jan 18 2016 20:12
thanks
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 20:12
It would be cool, if you will check that everything is fine in the develop branch.
Matt Warren
@mattwarren
Jan 18 2016 20:13
BTW can you wait to release till tomorrow night? I just noticed that "Baseline" doesn't work when you run a benchmark as new BenchmarkRunner().Run<BenchmarkProgram>();, it only works via the BenchmarkSwitcher
Yeah, I'll take a look at the develop branch as well
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 20:13

can you wait to release till tomorrow night

Of course.

Matt Warren
@mattwarren
Jan 18 2016 20:14
thanks
BTW this (me messing up the merge) makes me think that we need some sort of C.I build
I used AppVeyor before, can you access https://ci.appveyor.com/project/mattwarren/minibench-warren and see it?
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 20:16
It is a very good idea.
Matt Warren
@mattwarren
Jan 18 2016 20:16
Okay, I'll add an issue to track it
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 20:17

can you access https://ci.appveyor.com/project/mattwarren/minibench-warren and see it?

Seems that is is a public page, everyone can see it.

Matt Warren
@mattwarren
Jan 18 2016 20:18
okay, that might be because I'm on the free plan, I guess that doesn't matter for us does it? We could even put a badge on our GitHub page, showing the build status!!
Andrey Akinshin
@AndreyAkinshin
Jan 18 2016 20:20
Yep.