Conversation
|
🚀 Deployed on https://6922632911316ef2de0a7c2e--opengeos.netlify.app |
There was a problem hiding this comment.
Pull request overview
This pull request updates all MapLibre map instantiations in example notebooks to use the simpler anymap.Map class name instead of anymap.MapLibreMap. The Map class is an alias for MapLibreMap defined in the package's __init__.py, making it the recommended default map class for users.
- Simplified class name from
anymap.MapLibreMaptoanymap.Mapacross all examples - Updated notebook metadata (Python version and kernel display name) in
authorization_headers_example.ipynb
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/examples/maplibre/to_html.ipynb | Updated two map instantiations to use anymap.Map instead of anymap.MapLibreMap |
| docs/examples/maplibre/authorization_headers_example.ipynb | Updated six map instantiations to use anymap.Map, commented out request_headers parameter in first example, and updated notebook metadata (Python version to 3.12.9 and kernel name to "geo") |
| docs/examples/maplibre/add_vector_editor_example.ipynb | Updated five map instantiations to use anymap.Map instead of anymap.MapLibreMap |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| " center=[-74.0060, 40.7128],\n", | ||
| " zoom=12,\n", | ||
| " request_headers={\"Authorization\": \"Bearer YOUR_TOKEN_HERE\"},\n", | ||
| " # request_headers={\"Authorization\": \"Bearer YOUR_TOKEN_HERE\"},\n", |
There was a problem hiding this comment.
The request_headers parameter has been commented out. This appears to be the first example demonstrating how to set authorization headers at map creation, but the key functionality is now commented out.
If this is intentional for the example to run without requiring a token, consider either:
- Keeping the parameter uncommented but adding a note in the markdown explaining it's optional
- Moving this to a different example and starting with Example 2 (setting headers after creation) as the first example
No description provided.