Log shows a wrong name for plot images

[simonech]

After running a benchmark with [RPlotExporter] I get zillions of jpg files, but I'm not sure all of them have the correct name.

First of all, the log in the console says:

// * Export *
  BenchmarkDotNet.Artifacts/results/MyBench.Sleeps-report.csv
  BenchmarkDotNet.Artifacts/results/MyBench.Sleeps-report-github.md
  BenchmarkDotNet.Artifacts/results/MyBench.Sleeps-report.html
  BenchmarkDotNet.Artifacts/results/MyBench.Sleeps-measurements.csv
  BuildPlots.R
  BenchmarkDotNet.Artifacts/results/MyBench.Sleeps-20190623-012647-boxplot.png
  BenchmarkDotNet.Artifacts/results/MyBench.Sleeps-20190623-012647-barplot.png

But the files stored in the artifacts folder are called MyBench.Sleeps-barplot.png and MyBench.Sleeps-boxplot.png (so, they don't have the timestamp).

[AndreyAkinshin]

@simonech thanks for the report!

[simonech]

@AndreyAkinshin I could have a look at the first, since it's an issue in the .net code.
The second is generated in the R script, and I've no idea how that works 😄

maybe it would be better to split the issue in two issues

[AndreyAkinshin]

@simonech

I could have a look at the first, since it's an issue in the .net code.

Great, thanks! It should be pretty easy to fix.

maybe it would be better to split the issue in two issues

OK. Do you want to create another issue for the second problem (and edit the current one)?

[simonech]

@simonech

I could have a look at the first, since it's an issue in the .net code.

Great, thanks! It should be pretty easy to fix.

Indeed... I'll do it

maybe it would be better to split the issue in two issues

OK. Do you want to create another issue for the second problem (and edit the current one)?

Yeah, I've created a new ticket with the second part of the issue (#1183)

[AndreyAkinshin]

Great, thanks! Waiting for a PR. =)

[marcnet80]

Hi, @AndreyAkinshin. Could you please provide steps to reproduce this issue?
Maybe this behaviour relates to this line: https://github.com/dotnet/BenchmarkDotNet/blob/master/src/BenchmarkDotNet/Exporters/ExporterBase.cs#L34

[AndreyAkinshin]

@marcnet80 just run any benchmark with the [RPlotExplorter] attribute.
Since RPlotExporter generates a lot of different plots (not only boxplot and barplot) and there is no a simple way to get real names of these files (this logic is controlled by the internals of BuildPlots.R), I suggest to print just *.png instead of specific names.

[marcnet80]

@AndreyAkinshin, all generates plots have png extension?

[AndreyAkinshin]

@marcnet80 yes.

[marcnet80]

Ok. I will create pull-request today. Thank you.

[adamsitnik]

Fixed a long time ago in #1217, closing