Ditto--it's confusing that `samples` is said to be of type "int" but takes a string as its default value.  It's more idiomatic to just use `None` as the default and explain in the documentation that if `samples` is `None` (read: unspecified) all datapoints are used.