naobservatory / p2ra

4 stars 1 forks source link

summarize: don't skip location and date information from the top-level #95

Closed jeffkaufman closed 1 year ago

jeffkaufman commented 1 year ago

Before this change, if you made a Variable with a location and then included additional location information on its inputs, only the latter would be considered by summarize_location (and the same with summarize_date).

This was originally included in #85 where it fixed a bug that Flu was triggering, but that PR is taking a long time for review, so pulling it out for separate review (which I probably should have done from the start).