update write_loom to write layers

[VolkerBergen]

No description provided.

[codecov]

Codecov Report

Merging #53 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
- Coverage   64.94%   64.89%   -0.05%     
==========================================
  Files           9        9              
  Lines         716      715       -1     
==========================================
- Hits          465      464       -1     
  Misses        251      251

Impacted Files	Coverage Δ
anndata/readwrite/write.py	69.38% <100%> (-0.16%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bbef0bc...12d2600. Read the comment docs.

[flying-sheep]

Could you please add a test validating that an AnnData object with layers can be correctly written and read again?

[VolkerBergen]

Sure, done.

[flying-sheep]

where? i mean in this PR’s branch. and please make it a separate test, not new asserts in an old one 😃

[falexwolf]

Yes @VolkerBergen, please add a test. I don't see where you have added it.

[flying-sheep]

he added it in another PR, but should add it in this one.

@VolkerBergen if you don’t want to do the whole checkout dance you can also go to the file on branch patch-3 online and edit it there

[flying-sheep]

OK, to fix python 3.5, you have to add

import pytest
from importlib.util import find_spec
...
pytest.mark.skipif(find_spec('loompy') is None, reason="loompy not installed")
def test_readwrite_loom():
   ...

and for 3.6… apparently loompy now depends on matplotlib but don’t mention that.

either wait until linnarsson-lab/loompy#68 is fixed or add matplotlib to requirements-test.txt

[flying-sheep]

looks great, thank you!