That's OK, but I usually add an extra parens in the ternary case for clarity.
```
colunit = ('' if column.units is None else column.units)
```