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

[Feature] Allow usage of other options when --config/-c option is used with those other options missing #17

Open
lmpham1 opened this issue Oct 7, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@lmpham1
Copy link
Owner

lmpham1 commented Oct 7, 2021

Problem

Currently, the program does not support using --config/-c option with other options, and uses default setting if other options are not specified in the .json file

Expected Feature

The user should be able to specify other command-line options alongside with --config/-c, if those options are not already specified inside the config JSON file

Example

Given the following config.json file:

{
  "output": "/test/"
}

If the user only uses --config/-c, i.e.:

python main.py --config config.json

The program will produce an error since there are no input files/folders.

The user should be able to use other options alongside the --config/-c flag, so that they can specify any options missing from the config JSON file, i.e.:

python main.py --config config.json -i test.txt --lang en-UK

If both exists, the configs inside the config JSON file will take precedence over the command-line options

@lmpham1 lmpham1 added the enhancement New feature or request label Oct 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant