Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1156)

Issue 336960043: For phase3: 3 path from Oregon to N.Virginia

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 week, 6 days ago by Wei Wang
Modified:
1 week, 1 day ago
Reviewers:
dkostic, kirill.sc
CC:
manu_chaud, zhubiwen509, brahim Al-Qaysi, yulianlluo, farzad2050
Visibility:
Public.

Description

For phase3: 3 path from Oregon to N.Virginia Same problem: "result.png" cannot be seen on Code review page. please try to scan it in our git repository.

Patch Set 1 #

Patch Set 2 : create bidirectional US link #

Unified diffs Side-by-side diffs Delta from patch set Stats (+44 lines, -772 lines) Patch
A + Bidirectional_US_link/.DS_Store View 1 Binary file 0 comments Download
A + Bidirectional_US_link/Bidirection/.DS_Store View 1 Binary file 0 comments Download
A + Bidirectional_US_link/Bidirection/Trace1219.xml View 1 1 chunk +22 lines, -0 lines 0 comments Download
A + Bidirectional_US_link/Bidirection/index.html View 1 5 chunks +26 lines, -13 lines 0 comments Download
A + Bidirectional_US_link/Bidirection/jquery-3.2.1.min.js View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A Bidirectional_US_link/Bidirection/result.png View 1 Binary file 0 comments Download
A Bidirectional_US_link/Readme View 1 1 chunk +4 lines, -0 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/.DS_Store View 1 Binary file 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/Trace12.xml View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/index.html View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/jquery-3.2.1.min.js View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/request_Google_Elevation_API/.DS_Store View 1 Binary file 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_1.xml View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_2.xml View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_3.xml View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_all.xml View 1 0 chunks +-1 lines, --1 lines 0 comments Download
A + Bidirectional_US_link/Unidirection_Oregon_to_N.Virginia/result.png View Binary file 0 comments Download
D Oregon_to_N.Virginia/.DS_Store View 1 Binary file 0 comments Download
D Oregon_to_N.Virginia/Trace12.xml View 1 1 chunk +0 lines, -30 lines 0 comments Download
D Oregon_to_N.Virginia/index.html View 1 1 chunk +0 lines, -261 lines 0 comments Download
D Oregon_to_N.Virginia/jquery-3.2.1.min.js View 1 1 chunk +0 lines, -4 lines 0 comments Download
D Oregon_to_N.Virginia/request_Google_Elevation_API/.DS_Store View 1 Binary file 0 comments Download
D Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_1.xml View 1 1 chunk +0 lines, -84 lines 0 comments Download
D Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_2.xml View 1 1 chunk +0 lines, -68 lines 0 comments Download
D Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_3.xml View 1 1 chunk +0 lines, -84 lines 0 comments Download
D Oregon_to_N.Virginia/request_Google_Elevation_API/trace12_all.xml View 1 1 chunk +0 lines, -236 lines 0 comments Download
D Oregon_to_N.Virginia/result.png View Binary file 0 comments Download

Messages

Total messages: 2
Wei Wang
1 week, 6 days ago (2017-11-08 22:02:47 UTC) #1
Wei Wang
1 week, 1 day ago (2017-11-13 15:26:16 UTC) #2
On 2017/11/13 16:02:47, Wei Wang wrote:

The result of "Unidirection_Oregon_to_N.Virginia" is more readable because of
few amount of paths.
At the start of phase3, it's much better to test on this one.
However, we finally need to apply Dijkstra's algorithm on the Bi-direction one,
which has more points.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld c59f3c4