Skip to content

Conversation

MSameerAbbas
Copy link
Contributor

@MSameerAbbas MSameerAbbas commented Jul 5, 2025

Modified the image_check function to add the "zoom" to the options argument of kwarg_check. This is because in mapillary.interface, the following functions allow for zoom as a keyword argument:

  1. get_image_looking_at_controller
  2. get_image_close_to_controller

The first line of these functions use image_check to filter through only the correct keys. Before the change, if zoom was supplied, image_check would throw an InvalidKwargError exception in both these functions.

Modified the image_check function to add the "zoom" to the options argument of kwarg_check. This is because in mapillary.interface, the following functions allow for zoom as a keyword argument:
1. get_image_looking_at_controller
2. get_image_close_to_controller
The first line of these functions use image_check to filter through only the correct keys. Before the change, if zoom was supplied, image_check would throw an InvalidKwargError exception in both these functions.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 5, 2025
@nickplesha
Copy link
Contributor

Looks good, merging!

@nickplesha nickplesha merged commit e14ff00 into mapillary:main Jul 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants