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: create ConfigurableParameter() class #47

Open
ba58smith opened this issue Jun 17, 2019 · 0 comments
Open

feature: create ConfigurableParameter() class #47

ba58smith opened this issue Jun 17, 2019 · 0 comments

Comments

@ba58smith
Copy link
Collaborator

From a Slack conversation:
jkoz [1:41 PM]
Re: configurable value: Matti - you have this "ObservableValue" object you used for the individual GPS values (and some other things IIRC). That concept - have the value wrapped by a class, is very close to an idea I had: We could make a "ConfigurableParameter" class that is much like ObservableValue, but it could also supply itself as json, its own title etc. If we did that, then we could probably get away with just declaring the values you want to use in a Transform as a "ConfigurableParameter", and then all the other code (get_configuration(), set_configuration(), get_json_schema()) would work with that. That would prevent having to write the marshalling code over and over again.

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

No branches or pull requests

1 participant