Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate Handlers in CA library to directly consume the data sources (remove dependency on Aggregators) #243

Open
5 of 11 tasks
gfmatthews opened this issue Jan 11, 2023 · 1 comment

Comments

@gfmatthews
Copy link
Member

gfmatthews commented Jan 11, 2023

Description

As a Carbon Aware Developer, I want the datasource handlers (EmissionHander, ForecastHandler) to directly call the datasource and remove the overhead of having Aggregators in between so that the code is cleaner and extensible.
This would prevent from creating a new Aggregator layer every time a new data source interface is created.

Acceptance Criteria

  • Ensure that the CA library consumes the Datasources directly.
  • Remove reference to the Aggregator project
  • Mark Aggregator methods with 'deprecated' flag
  • Unit Tests are completed, and the code passes tests
  • Update Documentation, including Architecture diagrams, if needed

Dependencies

Task List

  • List of tasks (punch list) needed to complete the user story

Sprint-Ready Checklist

  • Acceptance criteria defined
  • Acceptance criteria are verifiable/testable
  • External / 3rd Party dependencies identified
  • Engineering team understands acceptance criteria
  • Is the user story small enough to be implemented in a short amount of time, but large enough to provide value?
@juzuluag
Copy link

PR: Green-Software-Foundation#253

vaughanknight added a commit that referenced this issue Feb 6, 2023
Added support for Markdown linter Github Action
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants