[GraphHopper] Small patch for distance calculations in CH and translation to spanish

[Peter K]

Hi Francisco,

this should not be necessary. Would you create some minimal reproducing
code (and put it e.g. on gist.github.com) where you see this problem?

Regards,
Peter.


> Hi Peter.
>
> Sorry, I should explain it a bit more.
>
> When I use this code:
>
> GHResponse ph = calculateRoute(flag1, flag2, gh);
>         System.out.println("Dist = " + ph.getDistance() + " Time: " +
> (t2-t3));
>
> ph.getDistance returns much more than the real distance. I create the
> graph with real distances in meters:
>
> EdgeIterator edgeIt = graph.edge(idNodo1, idNodo2,
> dist.doubleValue(), true);
> edgeIt.setName(name);
>
> With this change, it seems the distance is ok. 
>
> Maybe I am doing something wrong, or not in the last revision of the
> library.
>
> Best regards.
>
> Fran.
>
>
>
> 2013/7/19 Peter K <peathal at yahoo.de <mailto:peathal at yahoo.de>>
>
>     Hi Francisco,
>
>     what problems do you have?
>
>     Regards,
>     Peter.
>
>>     Hi Peter and all.
>>
>>     I have a little patch to solve some problems I have with distance
>>     calculations when using CH. I don't know if it happens only to
>>     me, but... just in case.
>>
>>     The change is simple, in Path4CH file:
>>     double dist = mainIter.getDistance(); // in repository =>
>>     calcDistance(mainIter);
>>
>>     when mainIter is NOT a shortcut, getDistance directly.
>>
>>     Hope it helps, and thank you very much for this excellent library!!.
>>
>>     Fran.
>>
>>
>>     diff --git
>>     a/core/src/main/java/com/graphhopper/routing/ch/Path4CH.java
>>     b/core/src/main/java/com/graphhopper/routing/ch/Path4CH.java
>>     index fb4f440..50199f4 100644
>>     --- a/core/src/main/java/com/graphhopper/routing/ch/Path4CH.java
>>     +++ b/core/src/main/java/com/graphhopper/routing/ch/Path4CH.java
>>     @@ -60,7 +60,7 @@
>>          {
>>              if (!mainIter.isShortcut())
>>              {
>>     -            double dist = calcDistance(mainIter);
>>     +            double dist = mainIter.getDistance();
>>     //calcDistance(mainIter);
>>                  distance += dist;
>>                  int flags = mainIter.getFlags();
>>                  time += calcTime(dist, flags);
>>     diff --git
>>     a/core/src/main/java/com/graphhopper/util/TranslationMap.java
>>     b/core/src/main/java/com/graphhopper/util/TranslationMap.java
>>     index f3c9dde..3ff3c12 100644
>>     --- a/core/src/main/java/com/graphhopper/util/TranslationMap.java
>>     +++ b/core/src/main/java/com/graphhopper/util/TranslationMap.java
>>     @@ -42,6 +42,19 @@
>>              de.put("turn %s", "%s abbiegen");
>>              de.put("turn %s onto %s", "%s abbiegen auf %s");
>>      
>>     +        TranslationHashMap es = new TranslationHashMap();
>>     +        translations.put("es", es);
>>     +        es.put("sharp left", "izquierda");
>>     +        es.put("sharp right", "derecha");        
>>     +        es.put("left", "izquierda");
>>     +        es.put("right", "derecha");
>>     +        es.put("slight left", "un poco a la izquierda");
>>     +        es.put("slight right", "un poco a la derecha");
>>     +        es.put("continue", "continue");
>>     +        es.put("continue onto %s", "siga por %s");
>>     +        es.put("turn %s", "gire por %s");
>>     +        es.put("turn %s onto %s", "gire %s por %s");
>>     +
>>              TranslationHashMap en = new TranslationHashMap()
>>              {
>>                  @Override
>>
>>     -- 
>>     Francisco José Peñarrubia
>>
>>     Software Colaborativo
>>     http://www.scolab.es
>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/graphhopper/attachments/20130719/4e5d285a/attachment.html>