Skip to content

v1.1.9 #332

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
thomaszwagerman wants to merge 29 commits into
base: main
Choose a base branch
from
Open

v1.1.9 #332

thomaszwagerman wants to merge 29 commits into from

Conversation

@thomaszwagerman
Copy link
Contributor

@thomaszwagerman thomaszwagerman commented Dec 14, 2025 •
edited
Loading

PolarRoute Pull Request Template

Date: 14/12/2025
Version Number: 1.1.9

Description of change

  • Update docs links.
  • Loosen MeshiPhi version strictness in requirements.
  • Account for SIC values being None, rather than absent in various vessel performance models.
  • Add Slocum and Boaty McBoatface vessel configs.
  • Add module logger, replaces logging.*() calls.
  • Enable the running of pytest in the main directory.
  • Reduce the amount of duplicate filepaths by adding dynamic discovery of relevant files needed for the tests.
  • Parameterise repetitive tests.
  • Mark smoothing tests as slow, enable running "not slow" tests.
  • Enable running of tests in parallel using pytest-xdist.

Fixes

Fixes #245, fixes #260, fixes #321, fixes #331

thomaszwagerman and others added 29 commits December 11, 2025 20:04
@thomaszwagerman
loosen meshiphi requirement
2e2f88d
@thomaszwagerman
account for SIC being None
2e937dd
@thomaszwagerman
Merge branch 'tz/sic_none' into tz/loosen_version
9f13852
@thomaszwagerman
also for alr
52e65d8
@thomaszwagerman
update docs links, remove html
5bde3df
@thomaszwagerman
add boaty and slocum configs
3013a77
@thomaszwagerman
introduce use of logging.getLogger() instead of logging.*
03a07b5
@thomaszwagerman
more logging to logger changes
8db1f00
@thomaszwagerman
remove unused imports
366a563
@thomaszwagerman
add dynamic filepath discovery for route and vessel tests
cba1ff7
@thomaszwagerman
move rogue import
ddcad71
@thomaszwagerman
Merge pull request #334 from bas-amop/tz/module_logger
2c072c3 
Replacing logging calls with module logger
@thomaszwagerman
add version number
a8fe494
@thomaszwagerman
Merge branch 'v1.1.9' of github.com:bas-amop/PolarRoute into v1.1.9
dfc8d95
@thomaszwagerman
further sic cellbox none instance
6d48a68
@thomaszwagerman
replace direct imports for wrapper functions
2b0fde8
@thomaszwagerman
consolidate several files into utils, remove unnecessary functions an…
a8a3e4f 
…d imports
@thomaszwagerman
also push utils
5629c75
@thomaszwagerman
fix utils
d796207
@thomaszwagerman
cut more unnecessary helper functions
6a57037
@thomaszwagerman
parametrize tests to cut repetition further
07a5354
@thomaszwagerman
shorten config validation, move ini setting to pyproject.toml
981e049
@thomaszwagerman
also parametrize unit tests
c88e5c5
@thomaszwagerman
move relevant text of testing_strategy.md to the docs
27c9f11
@thomaszwagerman
add developer section to index
3793c75
@thomaszwagerman
add pytest.mark.slow to route smoothing tests
4c3f352
@thomaszwagerman
add option to run tests in parallel with pytest-xdist
45d245f
@thomaszwagerman
remove mark from fixture
98177d4
@thomaszwagerman
Merge pull request #333 from bas-amop/tz/test_simplify
2252872 
Simplify testing suite
@thomaszwagerman thomaszwagerman marked this pull request as ready for review December 22, 2025 20:49
@thomaszwagerman thomaszwagerman mentioned this pull request Dec 26, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Ulvetanna Ulvetanna Awaiting requested review from Ulvetanna

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Define module level logger Update documentation links Testing strategy outdated Improve handling of NaN values

2 participants

@thomaszwagerman