While reviewing the demand generation code in src/InstanceGenerator.py, I noticed a logic bug in the enumerate_timed_commodities method where the index used to assign time flexibilities is never incremented.
In enumerate_timed_commodities, the index i is initialized to 0 but is never updated inside the for loop. This causes every candidate origin-destination pair to be assigned the exact same time flexibility (flexibleTime[0]). Is this sensible or do I miss something from the description?
While reviewing the demand generation code in src/InstanceGenerator.py, I noticed a logic bug in the enumerate_timed_commodities method where the index used to assign time flexibilities is never incremented.
In enumerate_timed_commodities, the index i is initialized to 0 but is never updated inside the for loop. This causes every candidate origin-destination pair to be assigned the exact same time flexibility (flexibleTime[0]). Is this sensible or do I miss something from the description?