Closed emdavis02 closed 3 months ago
The recent updates to the _make_provider_from_cli
function in sleap/nn/inference.py
enhance its capability to process multiple input and output paths from both CSV and text files. It now returns lists for added flexibility, alongside improved error handling. These changes streamline the workflow for users managing various data formats in the command-line interface, making data processing tasks more intuitive and robust.
Files | Change Summary |
---|---|
sleap/nn/inference.py |
Enhanced _make_provider_from_cli to support multiple input and output paths from CSV and text files, changing its return type and improving error handling and logic flow. |
tests/nn/test_inference.py |
Added tests for sleap_track functionality with CSV and text file inputs, validating expected behavior and handling of erroneous CSV inputs. |
docs/guides/cli.md |
Updated documentation to clarify acceptable input formats for data_path and expanded the output option to accept directory paths. |
sleap-track
to support text and CSV files as input for data_path
, which aligns directly with the changes implemented in this PR.π In the meadow where data plays,
A rabbit hops through code-filled days.
With paths anew, it bounds with glee,
Multiple inputs dance, wild and free!
Processing joy, in every line,
A CLI wonder, oh how divine! β¨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 88.88889%
with 11 lines
in your changes missing coverage. Please review.
Project coverage is 74.36%. Comparing base (
7ed1229
) to head (9e974fe
). Report is 25 commits behind head on develop.
Files | Patch % | Lines |
---|---|---|
sleap/nn/inference.py | 88.88% | 11 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
After having expanded sleap-track to support multiple videos inputs at once, we want to further simplify this process for projects with a large numbers of inputs.
Types of changes
Does this address any currently open issues?
1870
Outside contributors checklist
Thank you for contributing to SLEAP!
:heart:
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
data_path
and the-o OUTPUT
argument in the CLI documentation, enhancing user understanding.