Issue fix#158

[SARTHAK2427]

Fixes Issue #158

Changes:

Fixed incorrect loop definition in shipParamsPerDist.

Replaced hardcoded file paths in compare.py with dynamic relative paths.

Fixed AttributeError by replacing non-existent .fuel attribute with .fuel_rate

Further Details:

Summary

This PR fixes three issues affecting the execution and portability of the code. The second issue was discovered while addressing the first

1. Loop Definition Bug
   The loop in shipParamsPerDist incorrectly used len() with two arguments, which causes a TypeError in Python.
   Fix:
   Replaced the incorrect usage with a proper range() iteration so the loop correctly iterates over waypoint indices without runtime errors.

2. Hardcoded File Paths
   compare.py contained file paths that were hardcoded to the original developer’s local machine. This caused a FileNotFoundError when other users ran the script.
   Fix:
   Replaced these paths with relative paths using os.path, allowing the script to dynamically locate files based on the script’s directory. This makes the code portable and runnable on any system.

3. AttributeError (.fuel vs .fuel_rate):
   The script was attempting to access a non-existent .fuel attribute on the ShipParamsobject. Fix: Corrected to uses the proper .fuel_rate attribute.

Dependencies:

No new dependencies were introduced

PR Checklist:

In the context of this PR, I:

• have (already previously) filled the 52North Contributor License Agreement and received positive feedback on this matter
• [ X ] have filled the 52North Contributor License Agreement and am waiting for feedback
• [ X ] provide unit tests embedded in the WRT test framework (WeatherRoutingTool/tests) that allow the simple testing of the new/modified functionalities. All (previous and new) unit tests execute without new error messages.
• [ X ] ensure that the code formatter runs without errors/warnings
• [ X ] ensure that my changes follow the WRT’s guidelines for contributing at the time of the contribution