tinker checkpoint delete unexpected args

[mzio]

For tinker version 0.16.1, when trying to delete checkpoints via the checkpoint paths, e.g.,

tinker checkpoint delete tinker://5f2d7413-3980-502a-b012-9b7e122b3305:train:0/sampler_weights/final

we get the following error:

Usage: tinker [OPTIONS]
Try 'tinker --help' for help.

Error: Got unexpected extra argument (tinker://5f2d7413-3980-502a-b012-9b7e122b3305:train:0/sampler_weights/final)

Fwiw, deleting via --run-id works:

tinker checkpoint delete --run-id 5f2d7413-3980-502a-b012-9b7e122b3305:train:0

Will delete 2 checkpoint(s):
  - tinker://5f2d7413-3980-502a-b012-9b7e122b3305:train:0/weights/final (812.7 MB, created 1 week ago)
  - tinker://5f2d7413-3980-502a-b012-9b7e122b3305:train:0/sampler_weights/final (271.1 MB, created 1 week ago)

Total size: 1.1 GB
WARNING: This action is permanent and cannot be undone.
Are you sure you want to delete 2 checkpoint(s)? [y/N]: y

[Jah-yee]

Good day,

I have submitted a fix for this issue in PR #29.

The problem was the variadic argument (nargs=-1) which conflicted with Click's option parsing. Changed to a single optional argument checkpoint_path instead.

The command tinker checkpoint delete tinker://run-id/weights/0001 should now work correctly.

Thank you for your work on Tinker!

Warmly, RoomWithOutRoof

[Jah-yee]

Good day,

I noticed this issue regarding tinker checkpoint delete not accepting checkpoint paths as arguments. The CLI currently requires using --run-id to specify the checkpoint, but users expect to pass the full checkpoint path (e.g., tinker://5f2d7413-3980-502a-b012-9b7e122b3305:train:0/sampler_weights/final) directly.

Here's a suggested fix by adding positional argument support in the checkpoint delete command:

# In cli/checkpoint.py - add path as positional argument
@click.argument("path", required=False)
@click.option("--run-id", "run_id", ...)
@cmd.command("delete")
def delete(path: str, run_id: str, ...):
    if path:
        # Parse tinker:// URI and extract run_id
        run_id = parse_checkpoint_path(path)
    # existing logic...

This would allow both:

• tinker checkpoint delete --run-id 5f2d7413-3980-502a-b012-9b7e122b3305:train:0
• tinker checkpoint delete tinker://5f2d7413-3980-502a-b012-9b7e122b3305:train:0/sampler_weights/final

Thank you for your work on Tinker!

Warmly,
RoomWithOutRoof