Just a small nit here, can we call it report_data? I think it would be a good convention to add _data suffix if something is just a set of attribs, not a constructed object (i.e. Report).