re-write the cli using click (or maybe typer?) #63
Comments
|
Hi. Adding my 2 cents: maybe https://github.com/google/python-fire can help speed up the process :) |
|
I use docopt for most of my projects now as click is usually overkill for what I need. Based on what I see I think click is probably going to be your best option for your project as it has great modularity. |
|
@rmallof @RackReaver I'm not familiar with the two packages but thanks for the suggestion. Yes I think click is a good choice for igel. Maybe typer would be even better but I'm not sure. I must check it out but it is also based on click and has similar syntax so that's the advantage |
|
I'm not super familiar with ML but I'd be willing to give converting this a whirl. Do you have existing tests for the cli to ensure nothing is missed? |
|
@RackReaver At the moment there are no tests for the cli, but even then, it must be updated/adapted to the click syntax. I implemented the first simple version using argparse, which had limitations and this is exactly one of the reasons why I want to switch to click. If you want to give it a try, I would happily review and eventually update your PR. You don't have to be familiar with ML, the cli code has nothing to do with ML, it's just python. |
|
Hi @nidhaloff, I am new to open source and would like to start working on it. I would try to replace argparse with click in cli.py |
Description
I'm the creator and only maintainer of the project at the moment. I'm working on adding new features and thus I would like to let this issue open for newcomers who want to contribute to the project.
Basically, I wrote the cli using argparse since it is part of the standard language already. However, I'm starting to rethink this choice because it has some issues that the click library already overcome.
With that said, it would be great to re-write the cli in click or even in typer, which also uses click under the hood but adds more features.
If someone wants to work on this, please feel free to start directly, you don't need to ask for permission.
PS: Feel free to suggest other libraries. I just suggested click since I'm familiar with it
I hope not but If this issue stayed open for a long time, then I will start working on it myself
The text was updated successfully, but these errors were encountered: