-
Notifications
You must be signed in to change notification settings - Fork 11
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 example CLI #82
Add example CLI #82
Conversation
Looks good so far! |
One suggestion– could you put all the files into a |
Done. Looking forward to when this is fully functional and ready. 😄 |
BTW, I was not sure what other input parameters are required for collage (for example, the radius and binning), so those could also part of the CLI. |
@sarthakpati What are you expecting in the output CSV file, and what format would be best for CaPTk to process the features? Would you like our CLI tool to write out image files? |
Hi @nathanhillyer,
|
@sarthakpati Thanks for the feedback, and starter project. We're going to use #87 as the final pull request. Feel free to try it out via |
Hi @nathanhillyer, would it be possible for you to merge my commites into your PR? I want my contribution to be recognized in this effort. |
Yes, sure! |
Hi @nathanhillyer, I have merged your cli branch with mine and this is now up-to-date: https://github.com/sarthakpati/collageradiomics/tree/add_cli If you don't mind merging this, it would be great, thanks. |
Following conversation with Pallavi, I have added a basic command line interface to get collage features. Items that need to be populated are commented with
todo
.