-
Notifications
You must be signed in to change notification settings - Fork 43
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 CLI functionality #197
Comments
Adding CLI functionality to the package is very very easy and will not require a different package (as suggested on #186). It pretty much just requires one more module and some boilerplate in |
How do parameters work, given that each method has different ones? Still |
Good question. It won't be hard to make the CLI commands accept a variable number of parameters using
when both |
I guess that can be our post-NetSci breaking change. |
Flagging that this almost certainly can't be developed at the same time as #174. |
Yes let's solve #174 first
…On Wed, Jun 12, 2019 at 2:38 PM Stefan McCabe ***@***.***> wrote:
Flagging that this almost certainly can't be developed at the same time as
#174 <#174>.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#197?email_source=notifications&email_token=AAILYADKVGRKJBVJDCN46O3P2E7DTA5CNFSM4HOPJSUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXRM4EA#issuecomment-501403152>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAILYAENODROLIPBJW3RPLTP2E7DTANCNFSM4HOPJSUA>
.
--
www.leotrs.com | leo@leotrs.com
PhD student at the Network Science Institute, Northeastern University
|
Examples of desired usage:
The text was updated successfully, but these errors were encountered: