FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add a `DatetimeRange` dataclass to pass pairs from/to · Issue #15 · frequenz-floss/frequenz-client-dispatch-python · GitHub

Repository navigation

Add a DatetimeRange dataclass to pass pairs from/to #15

Open

Description

What's needed?

add a DatetimeRange dataclass to pass pairs from/to.

Proposed solution

No response

Use cases

No response

Alternatives and workarounds

No response

Additional context

No response

Activity

  1. added
    part:❓We need to figure out which part is affected
    priority:❓We need to figure out how soon this should be addressed
    type:enhancementNew feature or enhancement visitble to users
    on Feb 19, 2024
  2. llucax commented on Feb 19, 2024

    Contributor

    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?

  3. nkrtzr commented on Feb 20, 2024

    @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?

  4. 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
  5. 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
  6. 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.

  7. added
    priority:lowThis should be addressed only if there is nothing else on the table
    scope:breaking-changeBreaking change, users will need to update their code
    and removed
    priority:❓We need to figure out how soon this should be addressed
    on Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    part:clientAffects the client codepart:❓We need to figure out which part is affectedpriority:lowThis should be addressed only if there is nothing else on the tablescope:breaking-changeBreaking change, users will need to update their codetype:enhancementNew feature or enhancement visitble to users

    Type

    No type

    Fields

    Priority

    None yet

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL