add TODO

validate parameters