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

feat(delivery-dispatch): SDD dispatcher with location caching #127

Merged
merged 48 commits into from
Apr 20, 2022

Conversation

sperka
Copy link
Contributor

@sperka sperka commented Apr 20, 2022

Issue #, if available: #110

Description of changes:

  • generalized DispatchService for reusability in different domains
  • dispatch results/assignments contain delivery segments, all parameters in unit-value pairs throughout all domains
  • using SSM to access cloud resources - only SSM parameter keys and SecretsManager service keys are stored in application.properties. This resolves the tech debt of updating the properties file from CDK.
  • created a distance cache utility (CLI app) to import/export cache generated/using with h3
  • treat locations on domain-level, but use common base (see Location class in different domains)
  • tweak Graphhopper configs to use higher network size and reduce routing errors
  • add same-day delivery domain with direct pickup-dropoff, where vehicles are considered to leave/arrive from hubs, but goods are picked up from customer/supplier and delivered to order dropoff
  • add routing information in result objects using polylines

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…ion to use LocationBase in distance matrix lookups
…ution consumer and logger, and add constraints
@sperka sperka requested a review from mirgj April 20, 2022 09:44
@mirgj mirgj merged commit adef08f into main Apr 20, 2022
@mirgj mirgj deleted the feat/110-sdd-dispatcher branch April 20, 2022 09:52
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