Fix stacked_violin plot bug

[outlace]

num_rows was incorrectly calculated when swap_axes=False in the stacked_violin plot. This fixes issue #405

[falexwolf]

Thank you for this! Unfortunately, it breaks the tests. It's such a small change, but I guess @fidelram had something in mind when setting it the way he did...

[outlace]

I’ll have another look. But it doesn’t work the way it is now either

[fidelram]

@outlace I will check the problem with the test and integrate your changes in a new PR that addresses #512 and #524 if this is OK with you.

[outlace]

@fidelram Sounds good!

[fidelram]

@outlace Curiously, your change causes an error. Without your change I can run the tests correctly without a problem. I remember that I fixed a bug similar to this one that was recently integrated into master (see https://github.com/theislab/scanpy/pull/425/files#diff-b5175ed1415cdbf853646e523cbe8ae0L902). Could it be that you didn't have the latest pull from scanpy and that was causing the error?

[outlace]

Possible, I will try pulling the latest master and see if I still get the problem

[fidelram]

@outlace do you think I can close this PR?

[ivirshup]

@fidelram, did that PR happen? I assume #405 got fixed but am not actually sure.

[fidelram]

yes 405 got fixed. The code was moved to _stacked_violin.py

[ivirshup]

Great, thanks!