Skip to content

Commit

Permalink
fix(propagation): use seconds instead of minutes
Browse files Browse the repository at this point in the history
Fixes #347
  • Loading branch information
ansoncfit committed Oct 31, 2017
1 parent f668a08 commit 222d603
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/main/java/com/conveyal/r5/analyst/TravelTimeComputer.java
Original file line number Diff line number Diff line change
Expand Up @@ -75,10 +75,10 @@ public void write (OutputStream os) throws IOException {
int x = target % request.width;
int y = target / request.width;

final int travelTimeMinutes =
travelTimesToTargets[target] == FastRaptorWorker.UNREACHED ? FastRaptorWorker.UNREACHED : travelTimesToTargets[target] / 60;
final int travelTimeSeconds =
travelTimesToTargets[target] == FastRaptorWorker.UNREACHED ? FastRaptorWorker.UNREACHED : travelTimesToTargets[target];

output.accept(y * request.width + x, new int[] { travelTimeMinutes });
output.accept(y * request.width + x, new int[] { travelTimeSeconds });
}

output.finish();
Expand Down

1 comment on commit 222d603

@abyrd
Copy link
Member

@abyrd abyrd commented on 222d603 Nov 10, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ansoncfit looks to me like lines 78 & 79 are now equivalent to:
final int travelTimeSeconds = travelTimesToTargets[target]

I've refactored this in a feature branch.

Please sign in to comment.