Project

General

Profile

Actions

Task #4767

closed

NLSR prints the actual hyperbolic angle and radius to 4 decimal place.

Added by Saurab Dulal over 5 years ago. Updated over 5 years ago.

Status:
Abandoned
Priority:
Normal
Assignee:
Target version:
-
Start date:
11/03/2018
Due date:
% Done:

0%

Estimated time:

Description

Coordinates in conf file
cagliari: _ radius=13.5903793306882 angle=1.13934400000000

Coordinate from nlsr.log after nlsr reads it.
1541300268.965444 INFO: [nlsr.ConfParameter] Hyp R: 13.5904
1541300268.965461 INFO: [nlsr.ConfParameter] Hyp Angle 0: 1.13934

Coordinates in conf file
padua: _ radius=13.5903793306882 angle=1.13934426229508

Coordinate from nlsr.log after nlsr reads it.
1541300352.470658 INFO: [nlsr.ConfParameter] Hyp R: 13.5904
1541300352.470672 INFO: [nlsr.ConfParameter] Hyp Angle 0: 1.13934

If you see these angles between Cagliari and Padua, they are different in the conf file, but here NLSR rounded it to four digits which made the angles same. Thus, if the angle and radius are same, the distance calculation fails with the following message.

copelabs/log/nlsr.log:9706:1541300372.280126 WARNING: [nlsr.route.RoutingTableCalculator] Could not calculate hyperbolic distance from /ndn/padua-site/%C1.Router/cs/padua to /ndn/cagliari-site/%C1.Router/cs/cagliari

Same is happening for some additional nodes.

Actions #1

Updated by Saurab Dulal over 5 years ago

  • Priority changed from Normal to High
Actions #2

Updated by Ashlesh Gawande over 5 years ago

  • Subject changed from NLSR rounds the actual hyperbolic angle and radius to 4 decimal place. to NLSR prints the actual hyperbolic angle and radius to 4 decimal place.
  • Priority changed from High to Normal

Just need to specify the precision when printing.

We do need to define the difference needed between angles of nodes on same location more precisely. So we will avoid getting a diffTheta of zero.

Actions #3

Updated by Saurab Dulal over 5 years ago

  • Status changed from New to Abandoned
Actions

Also available in: Atom PDF