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

Apply keys and values from extra_vars to experiment configurations #229

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alexander-gorelik
Copy link

@alexander-gorelik alexander-gorelik commented Aug 24, 2021

@Lawouach
Copy link
Contributor

Hey @alexander-gorelik

As I mentioned in the issue, there is rationale for config keys to be full yexplicit. Otherwise, it can get complicated understanding where it came from when reading the experiment.

I think your change means it will implicitely inject config keys at runtime.

I think a larger approach is about how we can compose experiments, so you could define sections (like configs) in shared files and "include" them in experiments.

@alexander-gorelik
Copy link
Author

@Lawouach That is actually what we are doing.
Now it's not fully functional(in our point of view), it can only operate on existent keys in experiments forcing you to have all the keys you ever need in the exp file as empty values.

Regarding the explicit keys, it's correct but if someone chooses to use --var-file it is logical that he knows that his config comes from other files.

From one side it just increases the exp file with not required keys that you always need to remember to add for your injected config to work, from another side you will have better visibility from just looking at the exp file and seeing all the config in one place.

From our side, the former is more important because it reduces maintenance, copy-paste, and failed runs because someone forgot to add keys.

And we actually don't touch the config often after we finish writing the experiment.

@Lawouach
Copy link
Contributor

Right, I considered that one for a long time. While I'm still feeling a bit concerned about the ripple effect, I must admit I cannot quantify the risks of doing as you suggest.

So let's try how you propose because I can see the benefits. Let's try :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configuration files require the main experiment file to have the keys from the configurations.
2 participants