plotEnrichment fails on long sample names

[thomasmanke]

... because it cannot plot the within the default device sizes.

Possible solutions
i) trimming name
ii) reporting error properly (currently it says something like bottom >=top)
iii) optionally switch off plot (--> as for plotFingerprint in develop)

[dpryan79]

The only real options are:

1. Don't plot anything (already implemented for 2.5.1, which will hopefully come out next week friday)
2. Allow people to adjust the plot size, which we should probably do anyway since I just allowed that for almost everything else.

We have no way of determining what will actually fit on the plot, that's all up to matplotlib. I'll close this once I've implemented --plotWidth and --plotHeight.

[thomasmanke]

2. is implemented already, but users will have to know that this is the reason. The current error does not suggest it.

[dpryan79]

Oh, plotEnrichment, you can already change the width and height, though I can make the plot optional too.

[dpryan79]

This is now implemented in feature/implement256.

[dpryan79]

@thomasmanke There's no way for us to change the error message, it's not our code.

[dpryan79]

@thomasmanke This is now in the develop branch. Please module load deeptools/develop in a few minutes and try plotEnrichment without specifying a plot.