Should DataSet inherit from InformationEntity?

[Mrinal-Thomas-Epic]

In the source YAML, DataSet inherits from Entity, but in the docs diagram, it inherits from InformationEntity.

Which is correct? https://va-spec.ga4gh.org/en/latest/core-information-model/index.html

[mbrush]

Thanks Mrinal. It should inherit from InformationEntity. I tracked down this commit where it looks like Alex changed it to inherit from Entity - but there is no mention or rationale for this in the commit message.

I recall a while back there was concern about DataSet inheriting attributes it doesn't need from InformationEntity, but I think this concern was addressed after slimming down the InformationEntity class. If there is no objection, I can make a quick PR to change it back. @ahwagner @larrybabb please confirm.