Dino catalog/mohammad Attempt 1

The semantic meaning of this method, at first glance is "the string representation of genus". The fact that this method has side effects of modifying `@output` is surprising and probably undesirable. Also, methods with side effects are harder to test and work with. If you move to simply return a value, it'll be much cleaner code.

Also, this demonstrates one reason why "unknown" is a weird value. Instead of being able to do `"foo" if @genus`, you have an entire extra check.