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

simulation_property_list can only handle string parameter types #13

Open
jakobs opened this issue Feb 2, 2015 · 2 comments
Open

simulation_property_list can only handle string parameter types #13

jakobs opened this issue Feb 2, 2015 · 2 comments

Comments

@jakobs
Copy link
Collaborator

jakobs commented Feb 2, 2015

Currently when overriding properties with simulation_property_list, only string types are set. The code also doesn't check the type of an already existing parameter. At least the basic types which are defined in the cfg_manager should be supported (e.g. bool, int, double, string).

On another note: How can I set more complex types, e.g. vector or color?

@jakobs jakobs added the bug label Feb 2, 2015
@goldhoorn
Copy link
Collaborator

Regarding "Only string"
Other values than string was not used so far and therefore not implemented.

Regarding "more complex types":
the cfg manager does not support more complex types. I assume this would be a larger extension.

@jakobs
Copy link
Collaborator Author

jakobs commented Feb 2, 2015

If you set a parameter that is of another type, it is silently ignored (since the type is kept from the previous creation of the parameter).
With the more complex types, this is something I saw when loading yaml configurations. But I guess its not handled by the cfg_manager yet. Fair enough.

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

No branches or pull requests

2 participants