search

Praqma / memory-map-plugin

A repository for the memory-map-plugin
13 stars 16 forks source link

X-axis (build no) becomes very cluttered with a large amounts of builds. #39

Closed AndersHoglund closed 6 years ago

AndersHoglund commented 6 years ago

X-axis (build no) becomes very cluttered with a large amounts of builds. Sample usage here: http://andwho.sytes.net:8080/job/AndWho_Betaflight/ Or the the attached screen shot.

bf_jenkins_memmap

buep commented 6 years ago

Hi @AndersHoglund we do not run into the issue because we do not save that many builds.

I'm suspecting that other graphing plugins, like the Warnings plugin will do the same?

If we should fix this, we should maybe only write the build number for every x build when the number of builds increases to a certain number.

Would that be a solution? Should it be configurable or just a fixed configuration we do it?

AndersHoglund commented 6 years ago

You need to able to see the long term trend on metrics like this. No other plugins I use shows this problem, examples here: http://andwho.sytes.net:8080/job/BorisB_BetaFlight/ In some way builds numbers are scaled and show only every 5th, 10th or 50th value, depending on total range of build numbers.

AndersHoglund commented 6 years ago

And of course it should be a dynamic scaling, no more config items please. And certainly not a fixed scaling.

buep commented 6 years ago

Right, we will just use the same kind of scaling then.

Thanks.

buep commented 6 years ago

We can probably get this out with the changes in 40 (not linking to it).

OEHC commented 6 years ago

This issue should be fixed in commit 6351062 The x-axis is changed from being of class CategoryAxis3D from JFreeChart, to being of class ShiftedCategoryAxis which inherits from NoOverlapCategoryAxis, from hudson/util. From the javadoc of NoOverlapCategoryAxis:

This class implements X-axis label skipping algorithm to avoid drawing overlapping labels.

AndersHoglund commented 6 years ago

Thanks, pulled and built branch 39-x-axis-build-no-becomes-very-cluttered-with-a-large-amounts-of-builds . Seems to work OK and fixes this issue. jenikins-bff3-memory-usage-fixed