fix: multiple avoid problem by shaiguelman · Pull Request #417 · googlemaps/google-maps-services-python · GitHub

Fixes #382

Modified distance_matrix so that the parameter "avoid" accepts multiple values, such as "tolls|highways".

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.

What to do if you already signed the CLA

Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ️ Googlers: Go here for more info.

I have now signed the CLA, so that tag can be removed.

@shaiguelman do you need to add another label describing the pr to have it merged?

Thank you for offering this fix and for your patience. Let me know if you're willing to make these changes, or if you'd prefer me to submit a patch to your fork.

Please also add this parameter to the distance matrix test, which you can model off the test of the multi-argument parameter in test_directions.py

Add "indoor"

To mirror the implementation in directions.py, let's update this to list or string

In this case it wouldn't work with a list since we're calling the split function. Would you like me to add an if check before calling split?

add "indoor"

I think you mean to remove from avoid_tokens

No, this is meant to avoid repeats (i.e. "tolls | tolls"). So once it's already used one then it removes it from the list of valid tokens. I can add a comment to explain or remove it if you want repeats to be valid. Please let me know

We want to set it to the value of avoid_tokens right?

Here's where we could do parallel implementation with directions.py.
How about:

params["avoid"] = convert.join_list("|", avoid_tokens)

We could, just seems redundant to rejoin the string instead of just using the version that was never split

@wangela Sorry for the late response. Haven't checked this personal github account in a while. Added "indoor" option and replied to your other comments

shaiguelman added 2 commits September 5, 2021 20:49

Fixed avoid to allow multiple avoid parameters.

96039e5

Changed documentation to account for change.

a71dedb

blunderbuss-gcf Bot assigned wangela Sep 6, 2021

google-cla Bot added the cla: no This human has *not* signed the Contributor License Agreement. label Sep 6, 2021

Yakopoto mentioned this pull request Oct 4, 2021

fix: multiple avoid distance matrix #382 #420

Closed

4 tasks

google-cla Bot added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Nov 4, 2021

shaiguelman changed the title Fix multiple avoid problem Fix: multiple avoid problem Jan 29, 2022

shaiguelman changed the title Fix: multiple avoid problem fix: multiple avoid problem Apr 10, 2022

wangela requested changes Jan 25, 2023

View reviewed changes

wangela mentioned this pull request Jan 25, 2023

Multiple avoid problems #382

Open

shaiguelman added 2 commits May 11, 2025 00:35

Merge branch 'googlemaps:master' into master

e24bf3d

Add indoor

1b286e8

shaiguelman requested a review from wangela September 23, 2025 21:17

-                      indicated features. Valid values are "tolls", "highways" or "ferries".
+                      indicated features. Valid values are "tolls", "highways" or "ferries"
+                      as well as any combination of them separated by "|".
                   :type avoid: string

                   if avoid:
-                      if avoid not in ["tolls", "highways", "ferries"]:
-                          raise ValueError("Invalid route restriction.")
+                      valid_avoids = ["tolls", "highways", "ferries"]

+                      for token in avoid_tokens:
+                          if token not in valid_avoids:
+                              raise ValueError("Invalid route restriction.")
+                          valid_avoids.remove(token)

+                          if token not in valid_avoids:
+                              raise ValueError("Invalid route restriction.")
+                          valid_avoids.remove(token)
                       params["avoid"] = avoid

fix: multiple avoid problem by shaiguelman · Pull Request #417 · googlemaps/google-maps-services-python · GitHub

Uh oh!

fix: multiple avoid problem - #417

fix: multiple avoid problem#417shaiguelman wants to merge 4 commits intogooglemaps:mastergooglemaps/google-maps-services-python:masterfrom shaiguelman:mastershaiguelman/google-maps-services-python:masterCopy head branch name to clipboard

Conversation

shaiguelman commented Sep 6, 2021

Uh oh!

google-cla Bot commented Sep 6, 2021

What to do if you already signed the CLA

Individual signers

Corporate signers

Uh oh!

shaiguelman commented Sep 7, 2021

Uh oh!

kostyafarber commented Sep 22, 2022

Uh oh!

wangela left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

shaiguelman commented May 11, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

fix: multiple avoid problem#417
shaiguelman wants to merge 4 commits into
googlemaps:masterfrom
shaiguelman:master

wangela left a comment •
edited
Loading