Skip to content

Conversation

mrzzy
Copy link
Collaborator

@mrzzy mrzzy commented Oct 16, 2024

Motivation

Part of #6

Contents

Functionally completes ML pipeline:

  • adds from_*() factory methods to convert between Congestion, Rating & TrafficImage
  • extends TrafficImage to add processed_on, model_id fields needed by Rating.

Refactors:

  • reverts API client to previous implementation as the current done is missing fields from the API.
  • update API client to support TrafficImage.
  • update tests to use old API client.

Fixes:

  • Missing GCP project ID in DatabaseClient by hardcoding project ID.

@mrzzy mrzzy requested a review from devlinwaluja October 16, 2024 04:01
@mrzzy mrzzy marked this pull request as ready for review October 16, 2024 04:02
@mrzzy mrzzy merged commit 9b902b6 into master Oct 16, 2024
2 checks passed
@mrzzy mrzzy deleted the feat/ml-pipeline branch October 16, 2024 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant