quilt icon indicating copy to clipboard operation
quilt copied to clipboard

Add AnnData as format

Open flying-sheep opened this issue 1 year ago • 5 comments

Description

Add AnnData as supported format

TODO

  • [x] Check if AnnData can interact with buffers
  • [x] Add to optional feature dependencies (extras)
  • [x] Sample h5ad file
  • [x] Unit tests
  • [ ] Documentation
    • [ ] Python: Run build.py for new docstrings
  • [x] Changelog entry

flying-sheep avatar Aug 08 '22 11:08 flying-sheep

I think this looks good. I can’t run gendocs though, I get

Starting pydocmd_main...
Started generating documentation...
Traceback (most recent call last):
  File "/home/phil/Dev/Python/Misc/Quilt/gendocs/build.py", line 90, in <module>
    pydocmd_main()
  File "/home/phil/Dev/Python/Misc/Quilt/.venv/lib/python3.10/site-packages/pydocmd/__main__.py", line 251, in main
    loader.load_section(section)
  File "/home/phil/Dev/Python/Misc/Quilt/.venv/lib/python3.10/site-packages/pydocmd/loader.py", line 87, in load_section
    sig = get_function_signature(obj, scope if inspect.isclass(scope) else None)
  File "/home/phil/Dev/Python/Misc/Quilt/.venv/lib/python3.10/site-packages/pydocmd/loader.py", line 138, in get_function_signature
    sig = str(inspect.signature(function))
  File "/usr/lib/python3.10/inspect.py", line 3247, in signature
    return Signature.from_callable(obj, follow_wrapped=follow_wrapped,
  File "/usr/lib/python3.10/inspect.py", line 2995, in from_callable
    return _signature_from_callable(obj, sigcls=cls,
  File "/usr/lib/python3.10/inspect.py", line 2392, in _signature_from_callable
    raise TypeError('{!r} is not a callable object'.format(obj))
TypeError: <classmethod(<function Package.install at 0x7f29436355a0>)> is not a callable object

flying-sheep avatar Aug 19 '22 11:08 flying-sheep

Codecov Report

Merging #2974 (ab5c024) into master (2313e3a) will increase coverage by 0.12%. The diff coverage is 90.38%.

@@            Coverage Diff             @@
##           master    #2974      +/-   ##
==========================================
+ Coverage   35.14%   35.27%   +0.12%     
==========================================
  Files         679      679              
  Lines       29616    29658      +42     
  Branches     4400     4400              
==========================================
+ Hits        10410    10462      +52     
+ Misses      18013    18003      -10     
  Partials     1193     1193              
Flag Coverage Δ
api-python 91.17% <90.38%> (+0.23%) :arrow_up:
catalog 8.87% <ø> (ø)
lambda 86.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
api/python/setup.py 0.00% <ø> (ø)
api/python/quilt3/formats.py 85.20% <86.48%> (+3.34%) :arrow_up:
api/python/tests/test_formats.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov[bot] avatar Sep 09 '22 07:09 codecov[bot]

OK, done!

flying-sheep avatar Sep 09 '22 11:09 flying-sheep

those tests are pretty flaky, I didn’t touch lambda and now it fails …

flying-sheep avatar Sep 12 '22 14:09 flying-sheep

@sir-sigurd Can you please review the PR?

robnewman avatar Sep 26 '22 17:09 robnewman