Skip to content
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

ENH: Addition of ctk_cli #493

Closed
wants to merge 3 commits into from
Closed

Conversation

fbudin69500
Copy link
Member

ctk_cli [1] is a command line parsing python package that parses
arguments the same way SlicerExecutionModel does.
Using ctk_cli allows in Python scripts allows to have the same
user interface in Python or in C++.

[1] https://github.com/commontk/ctk-cli

ctk_cli [1] is a command line parsing python package that parses
arguments the same way SlicerExecutionModel does.
Using ctk_cli allows in Python scripts allows to have the same
user interface in Python or in C++.

[1] https://github.com/commontk/ctk-cli
@pieper
Copy link
Member

pieper commented May 12, 2016

Sounds cool - can you add a short example?

@fbudin69500
Copy link
Member Author

@pieper Sure. Do you have any preference where the example should go?

Template argument name was modified in template declaration but not
when using it in the templated function.
ctk_cli allows to parse python scripts command lines
the same way they would be parsed with
SlicerExecutionModel. This commit adds an example to
show how to use this package.
@fbudin69500
Copy link
Member Author

@pieper : I added an example in the extension examples. Let me know if that seems reasonable.

@pieper
Copy link
Member

pieper commented May 13, 2016

Yes, that makes sense - thanks!

@jcfr
Copy link
Member

jcfr commented May 16, 2016

Thanks for the patch.

Nitpick: In 80a5a9b, instead of modifying the files in Extensions/Testing. Could you:

Thanks
Jc

@fbudin69500 fbudin69500 mentioned this pull request May 16, 2016
@jcfr
Copy link
Member

jcfr commented May 19, 2016

Closing. Superseded by #497

@jcfr jcfr closed this May 19, 2016
@jcfr jcfr mentioned this pull request Mar 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants