To avoid two lookups, we can use this instead:
```
std::map<std::string, kinematics::KinematicsQueryOptions>::const_iterator it = kinematics_query_options_map_.find(group_name);
if (it == kinematics_query_options_map_.end())
  return false;
opt = it->second;
return true;
```