graphhopper / graphhopper/jsprit
ShipmentInsertionCalculator - JobInsertionContext does not clear RelatedActivityContext
- Dominant language
- Java
- Stars
- 1.8k
- Forks
- 641
- PR merge metrics
- No merged PRs in 30d
Description
In a delivery loop
calculator set **relatedActivityContext** (pickup information) and **activityContext** (delivery information)
in the next pickup loop, before delivery loop
we have situation that insertionContext has **activityContext** (new pickup information) and **relatedActivityContext** (previous pickup information)
So in any constraint if we use relatedActivityContext field we have wrong information there. I think null is expected when you check pickupShipment activity.
Is is a bug or there is any reason why we need previous pickup information in the **relatedActivityContext** field?
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by tracing ShipmentInsertionCalculator and JobInsertionContext through the delivery and pickup loops. Verify whether relatedActivityContext should be cleared before processing a pickup, and confirm the expected value when pickupShipment constraints inspect it; done means the context no longer exposes stale pickup information.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 45/100