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

Feature/6490 display the number of assigned tracks per flow in the flow table #564

Conversation

randy-seng
Copy link
Member

@randy-seng randy-seng commented Nov 15, 2024

OP#6490
Requires #558 to be merged first

Implemented the `GetRoadUserAssignments` use case to fetch road user assignments using repositories and assigner components. Added corresponding unit tests to validate the assignment retrieval process.
Implement `NumberOfTracksAssignedToEachFlow` use case to count assignments per flow. Introduce corresponding unit tests to ensure functionality and accuracy.
@randy-seng randy-seng requested a review from briemla November 15, 2024 21:38
@briemla briemla changed the base branch from main to task/5990-show-track-statistics November 19, 2024 11:39
Base automatically changed from task/5990-show-track-statistics to main November 19, 2024 12:12
Renamed the method `_tracks_assigned_to_flows` to `_initialize_flow_track_counts` to better reflect its function. This change improves readability and makes the code more intuitive for future maintenance.
@randy-seng randy-seng requested a review from briemla November 20, 2024 09:15
Replaced the standard dictionary with a defaultdict for initializing flow track counts. This simplifies the code and ensures that missing keys are assigned a default value of zero automatically.
…of-assigned-tracks-per-flow-in-the-flow-table' into feature/6490-display-the-number-of-assigned-tracks-per-flow-in-the-flow-table
@randy-seng randy-seng requested a review from briemla November 20, 2024 09:50
@briemla briemla merged commit 4b47ec7 into main Nov 20, 2024
6 checks passed
@briemla briemla deleted the feature/6490-display-the-number-of-assigned-tracks-per-flow-in-the-flow-table branch November 20, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants