NeurodataWithoutBorders / lindi

Linked Data Interface (LINDI) - cloud-friendly access to NWB data
BSD 3-Clause "New" or "Revised" License
4 stars 1 forks source link

Add method to retrieve keys in LindiH5pyFile class #87

Closed magland closed 4 months ago

magland commented 4 months ago

Fixes #86

bendichter commented 4 months ago

I confirm this works. Could you support client.items() as well?

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.46%. Comparing base (731fcb5) to head (24c5371). Report is 9 commits behind head on main.

Files Patch % Lines
lindi/LindiH5pyFile/LindiH5pyFile.py 75.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #87 +/- ## ========================================== + Coverage 79.43% 79.46% +0.02% ========================================== Files 30 30 Lines 2256 2269 +13 ========================================== + Hits 1792 1803 +11 - Misses 464 466 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

magland commented 4 months ago

I confirm this works. Could you support client.items() as well?

Yes this is now available. I have merged these into #84 and published as lindi 0.3.14.

(I am publishing that branch before it's actually merged into main because I need it for the newest neurosift/dendro stuff)