-
Notifications
You must be signed in to change notification settings - Fork 378
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
Validate argument names #805
Comments
As long as you don't run the script with the A quick example:
|
I managed to replicate the error in this minimal example. It happens when I add two different config files: config.json
config2.json
test.py
Now |
This looks like a bug. Thanks for reporting! I got down to: from sacred import Experiment
ex = Experiment('test')
ex.add_config({})
ex.add_config({})
@ex.automain
def main():
pass
|
I found the issue but I don't have a good solution for this yet. It is caused by the code around here: Lines 109 to 112 in 89d8bb1
preset , so it thinks that the fixed values are set in the first config scope. It then shows up in the 'modified' section of the summary returned by the second config scope (and not in added as it should be) and error checking code doesn't recognize it as an added argument.
|
Hello,
I found that if I call my script with incorrect argument names, no error is thrown.
E.g. calling
python train.py with "eopchs=5"
instead ofpython train.py with "epochs=5"
will not result in any error, but the script will run with an incorrect number of epochs.Is there any option to validate the arguments?
Thanks,
Ondrej
The text was updated successfully, but these errors were encountered: