Skip to content
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.

Sign up for GitHub

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

Jump to bottom

CLI Refactoring: Use jsonargparse #112

Open
juanmc2005 opened this issue Nov 8, 2022 · 0 comments
Open

CLI Refactoring: Use jsonargparse #112

juanmc2005 opened this issue Nov 8, 2022 · 0 comments
Labels
refactoring Internal design improvements that don't change the API

Comments

@juanmc2005
Copy link
Owner

Problem

The implementation of the CLI is a bit messy and mixed with the python API.

Idea

Use jsonargparse to group diart.stream, diart.tune and diart.benchmark into a single diart.py with automatic CLI documentation (i.e. removing argdoc.py).
The new API would remove the dot (diart stream instead of diart.stream), --help would be generated from the docstring, and a class Diart could contain the implementation of all three scripts.
@juanmc2005 juanmc2005 added the refactoring Internal design improvements that don't change the API label Nov 8, 2022
@juanmc2005 juanmc2005 added this to the Version 0.7 milestone Nov 8, 2022
@juanmc2005 juanmc2005 removed this from the Version 0.7 milestone Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactoring Internal design improvements that don't change the API
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@juanmc2005