-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add function to override default configuration #47
Comments
the path variable must be unique for the databases, because there can be several databases |
This suggestion reminds me a bit of a functionality i implemented some time ago (probably doesn't work out-of-the-box any more) |
For now, it is not a big problem since we can run local test without any change. But it is better to keep this issue open for the future. |
@felixbur https://stackoverflow.com/questions/48538581/argparse-defaults-from-file But before going further, I have some things in my head now:
Having these capabilities will improve the speed of experimenting with Nkululeko. Just use the arrow key from the terminal to recall the last commands (or history/ctrl-z) and edit the command line arguments to experiment with other variables to see their impact. |
so, 2 is also already implemented |
The example for INI file currently contain user-specific path, e.g.,
nkululeko/tests/exp_emodb_os_mlp.ini
Line 9 in 8581a0e
I want to request that the user can override the config without editing the config. The suggested CLI argument is
-o
or--override
. So the command will be,Benefits: No need to modify INI file for testing/trying Nkululeko.
Also, should the path variable be consistent across datasets? In that emodb the dataset path is named "emodb" but in other datasets (Android) is named "data".
The text was updated successfully, but these errors were encountered: