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

Miovision camera ids #1107

Merged
merged 13 commits into from
Jan 3, 2025
Merged

Miovision camera ids #1107

merged 13 commits into from
Jan 3, 2025

Conversation

gabrielwol
Copy link
Collaborator

@gabrielwol gabrielwol commented Dec 3, 2024

What this pull request accomplishes:

  • A new DAG to pull Miovision camera details & configuration dates from the API and insert into the database.
    • We sometimes need camera details to answer how many cameras are at each intersection.
    • Configuration details may be useful for determining when validation studies are relevant

Issue(s) this solves:

What, in particular, needs to reviewed:

What needs to be done by a sysadmin after this PR is merged

Change from dev_scripts to data_scripts

@gabrielwol gabrielwol self-assigned this Dec 3, 2024
@gabrielwol gabrielwol requested a review from leo-oxu December 3, 2024 20:16
@gabrielwol gabrielwol marked this pull request as ready for review December 3, 2024 20:21
@gabrielwol gabrielwol marked this pull request as draft December 4, 2024 21:45
@gabrielwol
Copy link
Collaborator Author

@leo-oxu Reverting this to draft, going to expand scope a bit

@gabrielwol gabrielwol marked this pull request as ready for review December 6, 2024 15:16
Copy link
Contributor

@leo-oxu leo-oxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gabrielwol for having me review it! Learnt a lot~

@gabrielwol gabrielwol merged commit e4f926e into master Jan 3, 2025
5 of 6 checks passed
gabrielwol added a commit that referenced this pull request Jan 3, 2025
gabrielwol added a commit that referenced this pull request Jan 3, 2025
@gabrielwol gabrielwol deleted the miovision-camera-ids branch January 3, 2025 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a new DAG to capture Miovision configuration lastupdated and camera details
2 participants