mir_eval.display.piano_roll should raise a ValueError when both pitches and midi are None

[craffel]

As specified here. There is no such check; it just throws a confusing AtributeError when None is passed to hz_to_midi. I think this is the only place this is true in display.