
Path: \nodexl70521\NodeXL\Layouts\Layouts\HarelKoren\HarelKorenFastMultiscaleLayoutInternal.cs
Line: 253 (maybe)
Code: " if (Math.Abs(C) < eps && Math.Abs(C) < eps) "
Should it be " if (Math.Abs(C) < eps && Math.Abs(E) < eps) "?



Well, there is clearly something wrong with the line of code you pointed out.
The person who wrote the HarelKoren layout, which is definitely one of the more complicated parts of NodeXL (it's over a thousand lines of dense calculations), is no longer around. I don't understand the algorithm myself, so I can't be 100% sure that your
revision is the correct one. Were you comparing NodeXL's code to another implementation when you discovered this? If so, is that what the other implementation did?
Thanks,
Tony



Hi Tony.
Thank you for your reply.
HarelKoren layout in NodeXL is the only code I've studied so far. As a fresh man, I'm not sure whether something's wrong with the code either. In my opinoin, it is only a minor mistake that doesn't matter in the normal cases. It is my shallow understanding
and just for your information.

