[BUG] Error: Got unexpected extra argument

[prakashjayy]

Describe the bug

Error: Got unexpected extra argument while calling a function using typer.

Write here a clear and concise description of what the bug is.

I have specified 3 arguments for a function and getting the above error while using typer. my code looks like this.

• main.py

import typer 
app = typer.Typer()

@app.command()
def generate_csv(
    cfg: str,
    csv_loc: str,
    weights_locs: str,
):
    typer.echo(cfg)
    typer.echo(csv_loc)

if __name__ == "__main__":
    app()

• run python infer.py generate-csv "xxx" "yyy" "lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt"

Error

Usage: infer.py [OPTIONS] CFG CSV_LOC WEIGHTS_LOCS
Try 'infer.py --help' for help.

Error: Got unexpected extra argument (lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt)

however this error disappears and it works normally if I add another function as below and wrap it with typer.

```python
import typer 
app = typer.Typer()

@app.command()
def test():
    print("working fine")

@app.command()
def generate_csv(
    cfg: str,
    csv_loc: str,
    weights_locs: str,
):
    typer.echo(cfg)
    typer.echo(csv_loc)

if __name__ == "__main__":
    app()

• run python infer.py generate-csv "xxxx" "yyyy" "lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt"

output:

xxxx  
yyyy

Environment

• I am using ubuntu 16.04 and Anaconda python 3.6 environment.
• I am using typer==0.3.0

[daddycocoaman]

When you only have one command, like in your first example, there is no need to specify the name of the command you want to run. Typer sees that you only have one command to run, therefore the user must be intending to run that command.

So your first example should just be python infer.py "xxx" "yyy" "lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt"

[prakashjayy]

Thanks @daddycocoaman.

[schenock]

When you only have one command, like in your first example, there is no need to specify the name of the command you want to run. Typer sees that you only have one command to run, therefore the user must be intending to run that command.

So your first example should just be python infer.py "xxx" "yyy" "lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt"

I don't think that's a good "design choice". Very confusing and weird. Why not support both cases when only 1 app.command wrapped func?

[JavierLopezT]

When you only have one command, like in your first example, there is no need to specify the name of the command you want to run. Typer sees that you only have one command to run, therefore the user must be intending to run that command.

So your first example should just be python infer.py "xxx" "yyy" "lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt"

This should be on the first page of the documentation

[zhipengwa]

When you only have one command, like in your first example, there is no need to specify the name of the command you want to run. Typer sees that you only have one command to run, therefore the user must be intending to run that command.
So your first example should just be python infer.py "xxx" "yyy" "lightning_logs/version_3/checkpoints/epoch=18-step=10164.ckpt"

I don't think that's a good "design choice". Very confusing and weird. Why not support both cases when only 1 app.command wrapped func?

I want to second this comment, it costs me entire day to figure out the root cause.

[samibtorres]

I had the same problem for a while and found the page where it says if there is only one command defined in the app, but still want to use its name, there needs to be a callback defined : https://typer.tiangolo.com/tutorial/commands/one-or-multiple/

[zanieb]

It'd be great so support both cases e.g. my_app and my_app my_command, I also was very confused by this during development of a prototype application. Could this be opened again?

[GiorgioSgl]

I lost a lot of a time to encounter the problem! I don't understand why this design choose. I agree with @zanieb support both cases is the best option.

[nguyenvulong]

I just learned Typer half an hour ago and found 2 terrible designs

• automatic conversion from underscore to hyphen (inherited from Click)
• this. Should have let us use python main.py func opt in case there's a single command.

https://github.com/BrianPugh/cyclopts fixed the latter with app.default() but did not "fix" the first one either.

[BrianPugh]

@nguyenvulong please open up an issue in the cyclopts repo for the first point; we can probably figure something out. Thanks!