I wonder if this could be useful for other teams, so if we should put it in api-common (if we make it directly a type of proto message) or base-client instead if we keep it as a python-only thing.
@frequenz-floss/api-team does anyone envision having to use datetime ranges in other APIs? I guess for reporting if you are requesting data for a range you'll also have to specify a range, right?
@frequenz-floss/api-team does anyone envision having to use datetime ranges in other APIs? I guess for reporting if you are requesting data for a range you'll also have to specify a range, right?
the weatehr client is using a list of datetime objects to specify the time of the forcasts to be filtered out.
But for now a list with explicit timestamps is implemented. I'm not sure if ranges in combinations of ranges and timedeltas could be of interest for further development. What do you think @cwasicki?
changed the title [-]add a DatetimeRange dataclass to pass pairs from/to.[/-][+]Add a `DatetimeRange` dataclass to pass pairs from/to.[/+]on Feb 20, 2024
changed the title [-]Add a `DatetimeRange` dataclass to pass pairs from/to.[/-][+]Add a `DatetimeRange` dataclass to pass pairs from/to[/+]on Feb 20, 2024
In my opinion for weather it can make sense to simplify the query to datetime ranges. Also, ranges are used in other APIs like reporting and I think trading.
Activity
llucax commented on Feb 19, 2024
I wonder if this could be useful for other teams, so if we should put it in api-common (if we make it directly a type of proto message) or base-client instead if we keep it as a python-only thing.
@frequenz-floss/api-team does anyone envision having to use datetime ranges in other APIs? I guess for reporting if you are requesting data for a range you'll also have to specify a range, right?
nkrtzr commented on Feb 20, 2024
the weatehr client is using a list of datetime objects to specify the time of the forcasts to be filtered out.
But for now a list with explicit timestamps is implemented. I'm not sure if ranges in combinations of ranges and timedeltas could be of interest for further development. What do you think @cwasicki?
cwasicki commented on Feb 21, 2024
In my opinion for weather it can make sense to simplify the query to datetime ranges. Also, ranges are used in other APIs like reporting and I think trading.