Skip to content

feat(glides): add revenue field to vehicle trip assignments ingestion#711

Merged
runkelcorey merged 3 commits intomainfrom
hh-add-revenue-to-glides-schema
Mar 6, 2026
Merged

feat(glides): add revenue field to vehicle trip assignments ingestion#711
runkelcorey merged 3 commits intomainfrom
hh-add-revenue-to-glides-schema

Conversation

@huangh
Copy link
Collaborator

@huangh huangh commented Mar 1, 2026

required from https://github.com/mbta/glides/pull/3251

updating glides schema to match upstream.

verified with glides-dev kinesis stream

@huangh huangh force-pushed the hh-add-revenue-to-glides-schema branch from 5db8fb3 to 27ab1f4 Compare March 1, 2026 22:40
@github-actions
Copy link

github-actions bot commented Mar 1, 2026

LCOV of commit 27ab1f4 during Continuous Integration (Python) #1785

Summary coverage rate:
  lines......: 62.1% (3409 of 5492 lines)
  functions..: 28.6% (261 of 914 functions)
  branches...: no data found

Files changed coverage rate:
                                                                                     |Lines       |Functions  |Branches    
  Filename                                                                           |Rate     Num|Rate    Num|Rate     Num
  =========================================================================================================================
  src/lamp_py/ingestion/glides.py                                                    |95.6%    181|45.5%    44|    -      0

@github-actions
Copy link

github-actions bot commented Mar 3, 2026

LCOV of commit 61320de during Continuous Integration (Python) #1786

Summary coverage rate:
  lines......: 61.9% (3401 of 5492 lines)
  functions..: 28.6% (261 of 914 functions)
  branches...: no data found

Files changed coverage rate:
                                                                                     |Lines       |Functions  |Branches    
  Filename                                                                           |Rate     Num|Rate    Num|Rate     Num
  =========================================================================================================================
  src/lamp_py/ingestion/glides.py                                                    |95.6%    181|45.5%    44|    -      0

@huangh huangh force-pushed the hh-add-revenue-to-glides-schema branch from 61320de to 0c9caa4 Compare March 5, 2026 11:04
@huangh huangh marked this pull request as ready for review March 5, 2026 11:07
@huangh huangh requested a review from a team as a code owner March 5, 2026 11:07
@huangh huangh requested a review from runkelcorey March 5, 2026 11:07
@github-actions
Copy link

github-actions bot commented Mar 5, 2026

LCOV of commit 0c9caa4 during Continuous Integration (Python) #1802

Summary coverage rate:
  lines......: 62.1% (3412 of 5497 lines)
  functions..: 28.6% (261 of 914 functions)
  branches...: no data found

Files changed coverage rate:
                                                                                     |Lines       |Functions  |Branches    
  Filename                                                                           |Rate     Num|Rate    Num|Rate     Num
  =========================================================================================================================
  src/lamp_py/ingestion/glides.py                                                    |95.6%    181|45.5%    44|    -      0

Copy link
Collaborator

@runkelcorey runkelcorey left a comment

Choose a reason for hiding this comment

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

I think this looks right.

@runkelcorey runkelcorey merged commit 8585f58 into main Mar 6, 2026
10 checks passed
@runkelcorey runkelcorey deleted the hh-add-revenue-to-glides-schema branch March 6, 2026 20:46
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.

2 participants