graphhopper / graphhopper/jsprit

ShipmentInsertionCalculator - JobInsertionContext does not clear RelatedActivityContext

Open
#575 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.