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

Detect valid parameters for a test dynamically #7

Open
2 tasks
HalosGhost opened this issue Dec 12, 2022 · 0 comments
Open
2 tasks

Detect valid parameters for a test dynamically #7

HalosGhost opened this issue Dec 12, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@HalosGhost
Copy link
Collaborator

Right now, the test controller must be manually updated to be aware of and modify new parameters which the system is capable of understanding in the config. At minimum, this means if a component learns of a new config option, and we want it to be controllable from the TC, that config option needs to be manually added into the TC and wait for it to be deployed.

There are kind of two ways we could improve this, one of which is a short-term workaround, the other is far better long-term.

  • Add an adhoc additional-parameter text-area
    • allow experimenter to arbitrarily add settings to the final config
  • Use something like docopt for each component
    • allow the TC to parse out options which are valid for a given test configuration (parsing the same docopt input as the components)
    • enables simplifying the UI to only show options which are relevant given the components present in the test, and to automatically adjust to parameters available given the code checked out for the specified commit

Probably we should actually do both of these (since the first option enables some more significant agility quickly and with minimal effort), but the second solution is far more effectively long-term and will minimize a lot of maintenance boilerplate for new developments.

cf. mit-dci/opencbdc-tx#81

@HalosGhost HalosGhost added the enhancement New feature or request label Dec 12, 2022
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