Split cells when neighbouring inaccessible cells

[gecoombs]

PolarRoute Pull Request

Date: 26/03 Version Number: 0.4.0

Description of change

Adds a method to the VesselPerformanceModeller that will split any cells neighbouring inaccessible cells (including land cells) to match the size of the inaccessible cell.

Fixes #264

Testing

Files Changed:

polar_route/vessel_performance/vessel_performance_modeller.py

• [x] My changes require one or more test files to be updated for all regression tests to pass.

Failing tests: vessel_test_results.txt

Updated test results: vessel_tests_updated.txt

Unit test results: vessel_unit_test_dump.txt

Checklist

• [x] My code follows pep8 style guidelines.
• [x] I have commented my code, particularly in hard-to-understand areas.
• [x] I have updated the documentation of the codebase where required.
• [x] My changes generate no new warnings.
• [x] My PR has been made to the 0.4.x branch (DO NOT SUBMIT A PR TO MAIN)

[gecoombs]

This PR causes various regression tests to fail as the additional splitting modifies the vessel performance mesh and its neighbour graph. I have not updated the test files on this branch yet as the final changes @Ulvetanna needs to make to resolve the issue with splitting around waypoints will also require changing the tests and it may be easier to roll all of these changes into one big update.

[gecoombs]

For comparison, images showing the difference in the vessel performance meshes for the grf_normal environment mesh from the regression test examples:

Reference mesh:

New mesh:

[gecoombs]

I have now modified the vessel performance regression test files to match this version of the code as any further changes related to waypoint splitting will only change the route planning test results.