[BUG] Using Tab-autocomplete from command-line-api (in MLNET CLI) not working properly

dotnet / command-line-api

Command line parsing, invocation, and rendering of terminal output.

https://github.com/dotnet/command-line-api/wiki

MIT License

3.37k stars 378 forks source link

[BUG] Using Tab-autocomplete from command-line-api (in MLNET CLI) not working properly #424

Closed CESARDELATORRE closed 5 years ago

CESARDELATORRE commented 5 years ago

@jonsequitur , @srsaggam

When using tab autocompletion provided by "command-line-api" after typing mlnet new, it always suggests you the same argument, in current MLNET CLI version it is always --dataset.

Here are the installed NuGets:

Here's the behavior: mlnet-tab-complete

That happens no matter if I previously wrote something like:

mlnet new --label-column-na --> Should suggest mlnet new --label-column-name

mlnet new --h --> Should suggest mlnet new --help

No matter what, it always suggests --dataset

jonsequitur commented 5 years ago

Does this also repro if you type mlnet new label-column-na⇥?

CESARDELATORRE commented 5 years ago

Yes, it happens the same thing, see below:

mlnet-tab-complete-02

srsaggam commented 5 years ago

Amazing! Thank you.

CESARDELATORRE commented 5 years ago

Thanks a lot! 👍

CESARDELATORRE commented 5 years ago

Btw, when is this fix available on the public NuGet feed? Is it available now?

srsaggam commented 5 years ago

@CESARDELATORRE I think its available in myget. I have tried out the latest package. There may be some other bug in here. I have set up a debug session with @jonsequitur sometime today.

jonsequitur commented 5 years ago

Typically a new version of the package is available in the MyGet feed a few minutes after a merge.