Skip to content

Making PolarRoute-server compatible with latest PolarRoute and MeshiPhi versions#175

Open
thomaszwagerman wants to merge 3 commits intomainfrom
tz/version_updates
Open

Making PolarRoute-server compatible with latest PolarRoute and MeshiPhi versions#175
thomaszwagerman wants to merge 3 commits intomainfrom
tz/version_updates

Conversation

@thomaszwagerman
Copy link
Copy Markdown
Collaborator

@thomaszwagerman thomaszwagerman commented Dec 14, 2025

Specifically updating how evaluate_route works with PolarRoute changes to route_calc api:

# From
calc_route = route_calc(route_file.name, mesh_file.name)

# To
calc_route = route_calc(df, from_wp, to_wp, mesh.json, "fuel")

Also removing all TempFile related code, as now longer required for new API.

@davidwyld
Copy link
Copy Markdown
Collaborator

I appreciate this is still in draft, but looks good so far, I'm very pleased to finally be getting rid of the need to deal with tempfiles now we can use the python interface more directly.

Feel free to make any other improvements to this function you feel are necessary.

I'm sure you will, but don't forget to restrict the polar-route version in the pyproject.toml from this point.

@thomaszwagerman thomaszwagerman marked this pull request as ready for review April 7, 2026 12:32
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

Coverage

Coverage Report
FileStmtsMissBranchBrPartCoverMissing
polarrouteserver
   __init__.py3000100% 
   _version.py11000100% 
   celery.py153000 80%
polarrouteserver/route_api
   __init__.py4000100% 
   models.py722000 97%
   responses.py33020100% 
   serializers.py12053644 94%
   tasks.py124212877 82%
   utils.py108104244 91%
   views.py276175244 94%
polarrouteserver/route_api/management
   __init__.py0000100% 
polarrouteserver/route_api/management/commands
   __init__.py0000100% 
   ensure_adminuser.py2222600 0%
   insert_mesh.py2626800 0%
polarrouteserver/utils
   __init__.py0000100% 
   loggers.py1010000 0%
TOTAL8241161741985% 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants