8000 Use rich to print inference CLI inputs and add more provenance by talmo · Pull Request #651 · talmolab/sleap · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Use rich to print inference CLI inputs and add more provenance #651

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Feb 13, 2022
Merged

Use rich to print inference CLI inputs and add more provenance #651

merged 1 commit into from
Feb 13, 2022

Conversation

talmo
Copy link
Collaborator
@talmo talmo commented Feb 13, 2022

Description

  • More readable CLI arg printing
  • Add all CLI args to provenance for better metadata
  • Print more useful message when models not provided

Types of changes

  • Bugfix
  • New feature
  • Refactor / Code style update (no logical changes)
  • Build / CI changes
  • Documentation Update
  • Other (explain)

Does this address any currently open issues?

[list open issues here]

Outside contributors checklist

  • Review the guidelines for contributing to this repository
  • Read and sign the CLA and add yourself to the authors list
  • Make sure you are making a pull request against the develop branch (not main). Also you should start your branch off develop
  • Add tests that prove your fix is effective or that your feature works
  • Add necessary documentation (if appropriate)

Thank you for contributing to SLEAP!

❤️

8000
@talmo
Use rich to print inference CLI inputs and provenance
8fb6722 
- Add all args to provenance for better metadata
- Print more useful message when models not provided
@codecov-commenter
Copy link

Codecov Report

Merging #651 (8fb6722) into develop (0d2423a) will decrease coverage by 0.00%.
The diff coverage is 16.66%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #651      +/-   ##
===========================================
- Coverage    64.89%   64.88%   -0.01%     
===========================================
  Files          124      124              
  Lines        20176    20182       +6     
===========================================
+ Hits         13093    13095       +2     
- Misses        7083     7087       +4
Impacted Files Coverage Δ
sleap/nn/inference.py 63.60% <16.66%> (-0.21%) ⬇️

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 0d2423a...8fb6722. Read the comment docs.

@talmo talmo merged commit 4d91680 into develop Feb 13, 2022
@talmo talmo deleted the talmo/richer-inference-cli branch February 13, 2022 05:59
@talmo talmo mentioned this pull request Feb 16, 2022
Merged
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@talmo @codecov-commenter
0