stulp / dmpbbo

Python/C++ library for Dynamical Movement Primitives and Black-Box Optimization
GNU Lesser General Public License v2.1
224 stars 90 forks source link

Improve error-handling of get_param_vector(_size) #75

Open stulp opened 2 years ago

stulp commented 2 years ago

If not trained, should an error be raised or just don't count it?

_Originally posted by @ignacio-pm in https://github.com/stulp/dmpbbo/pull/72#discussion_r945788466_

stulp commented 2 years ago

If calling set_selected_param_names is needed before get_param_vector, I would write more clear here

_Originally posted by @ignacio-pm in https://github.com/stulp/dmpbbo/pull/72#discussion_r945807755_

stulp commented 2 years ago

When calling get_param_vector or get_param_vector_size, error-handling behavior should be consistent and documented.