Skip to content

Conversation

tlambert03
Copy link
Member

closes #4

this PR adds a cli command ndv ... and attempts to read basically anything 😂 as per @alisterburt request :)

it's a work in progress... would be installed via pip install ndv[io] and would lean heavily on aicsimagieo (or bioio, not sure yet) as well as some additional stuff

Copy link

codecov bot commented Jun 9, 2024

Codecov Report

❌ Patch coverage is 0.90909% with 109 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.35%. Comparing base (52fcfc6) to head (11b53dd).

Files with missing lines Patch % Lines
src/ndv/io.py 0.00% 99 Missing ⚠️
src/ndv/cli.py 0.00% 10 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.90%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   86.11%   84.35%   -1.77%     
==========================================
  Files          46       48       +2     
  Lines        5214     5323     +109     
==========================================
  Hits         4490     4490              
- Misses        724      833     +109     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

@alisterburt alisterburt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Loooove it, just got off the plane but excited to give it a whirl! Thanks for the effort 🙂

src/ndv/io.py Outdated


def imread(path: str | Path) -> Any:
"""Just read the thing already."""

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😂

@tlambert03 tlambert03 mentioned this pull request Dec 4, 2024
@tlambert03 tlambert03 force-pushed the cli branch 2 times, most recently from 5f4c7b1 to 1e9fb46 Compare August 24, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: command line interface

2 participants