docs: add availability note to vessel_risk_level docstrings#563
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Python SDK documentation to clarify that the vessel_risk_level / vessel_risk_level_excluded filters may not be available for all accounts, and that when unavailable the API will ignore the filter and omit vessel_risk_level from response data. The same clarification is added to the affected example notebooks.
Changes:
- Add availability disclaimers to voyage endpoint
search()docstrings forvessel_risk_levelandvessel_risk_level_excluded. - Add the same disclaimers to vessel availability endpoint docstrings for
filter_vessel_risk_level/exclude_vessel_risk_level. - Update academy notebooks with a user-facing note about
risk_rating/risk_rating_exclmapping to the API fields.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 15 comments.
Show a summary per file
| File | Description |
|---|---|
| vortexasdk/endpoints/voyages_vessel_class_breakdown.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_top_hits.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_timeseries.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_timeseries_v2.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_search_enriched.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_routes_breakdown.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_product_breakdown.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_geography_breakdown.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/voyages_congestion_breakdown.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/vessel_availability_timeseries.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/vessel_availability_search.py | Docstring note added for risk level filter availability. |
| vortexasdk/endpoints/vessel_availability_breakdown.py | Docstring note added for risk level filter availability. |
| docs/examples/academy_use_cases/Freight & Voyages/Fleet distribution per location.ipynb | Notebook narrative updated with availability note for risk rating params. |
| docs/examples/academy_use_cases/Freight & Voyages/Case Study of Voyages data.ipynb | Notebook narrative updated with availability note for risk rating params. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
OllyDS
approved these changes
May 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CHANGELOG
vessel_risk_levelandvessel_risk_level_excludeddocstringsvessel_risk_levelwill notappear in response data
TESTS
COMMENTS