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

Add more fine-grained control of default values #133

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

Conversation

mheden
Copy link
Collaborator

@mheden mheden commented Sep 15, 2022

Add a new flag to the object dictionary that makes COE_initDefaultValues skip setting default values.

Usecase:
Say that you have a object dictionary that should contain common values between all boards on a product but certain elements are have different defaults (configuration parameters, s/n, etc).

The current stack only have a "all or nothing" approach with the configuration parameter skip_default_initialization so that is not useful here. It is possible to use the set_defaults_hook and override values after they have been set to the default value. However, it felt like a better solution to just protect certain elements instead.

Add a new flag to the object dictionary that makes `COE_initDefaultValues`
skip setting default values.
@nakarlsson
Copy link
Contributor

Say that you have a object dictionary that should contain common values between all boards on a product but certain elements >are have different defaults (configuration parameters, s/n, etc).

Doesn't that require a "set_defaults_hook" anyway? Either you use the hook or use skip default and do it explicit on your own?

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.

2 participants