We should remove arbitrary `start_utc - pd.Timedelta("5T")`

[simlmx]

We have an arbitrary timestamp adjustment that might make sense for one PV provider, but probably not in general.

PVConsumer/pvconsumer/save.py

Line 52 in 41972a4

pv_yield_df["start_datetime_utc"] = pv_yield_df["datetime_utc"] - pd.Timedelta("5T")

Perhaps we could add an argument to change it in save_to_pv_site_database, and it should default to 0 (less surprising than "- 5 min")?

This is for the "start" timestamp, and If I'm not mistaken, the "end" timestamp is also tweak later in the process. We should probably take care of that one at the same time.

[peterdudfield]

I've added a note in the code for now. https://github.com/openclimatefix/PVConsumer/blob/issue/54-live-tests/pvconsumer/save.py#L55
I hope thats enough for the moment

[simlmx]

Thanks! I'd tend to keep the ticket open until we properly fix it but that's probably good enough for now.