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.

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

Unify the --model argument in train.py #33

Open
faizansana opened this issue Apr 21, 2024 · 0 comments
Open

Unify the --model argument in train.py #33

faizansana opened this issue Apr 21, 2024 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@faizansana
Copy link
Owner

Currently, there are two arguments to pass in a model.

The --model is the name of the model while --model-path is for selecting a path to a model to be retrained. To unify, remove the model-path argument and test if the model argument is a path or a name of a model and handle it within script directly.

@faizansana faizansana added enhancement New feature or request good first issue Good for newcomers labels Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant