Since this is a special case, you don't want it to get lost later in the code. It might be nice to have a constant called something like `ALLOW_STRINGS` that's a list of metric types where we allow strings, so you'd have a cleaner:

```python
if metadata[1] in ALLOW_STRINGS:
```
That way we can extend it later and it's very readable. Seem reasonable?