-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
add slap-reproject to setup cfg #48
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #48 +/- ##
=======================================
Coverage 74.49% 74.49%
=======================================
Files 3 3
Lines 298 298
=======================================
Hits 222 222
Misses 76 76 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it not be sleap_anipose:triangulation.reproject_cli instead of sleap_anipose.triangulate:reproject_cli?
whoops, sorry! very confused as to why it still generated the reprojections despite that error... but I fixed it now and it runs! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
Running 'slap-reproject' resulted in an error where 'slap-reproject is not recognized as an internal or external command' which was identified in issue #46. This was because the reproject_cli was not added to the setup.cfg file in the console_scripts entry points. I have added slap-reproject to that list and the file is now being created as expected.