Skip to content

Conversation

Yakopoto
Copy link

@Yakopoto Yakopoto commented Oct 4, 2021

Sorry for creating this PR as second solution for #417. My team need this feature ASAP for critical function.

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #382

@google-cla
Copy link

google-cla bot commented Oct 4, 2021

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
Corporate signers

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added the cla: no This human has *not* signed the Contributor License Agreement. label Oct 4, 2021
@google-cla 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 Oct 4, 2021
@Yakopoto Yakopoto changed the title fix multiple avoid distance matrix #382 fix: multiple avoid distance matrix #382 Oct 4, 2021
Copy link

@chantyapk chantyapk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks nice one

@wangela wangela assigned wangela and unassigned arriolac Jan 25, 2023
@wangela
Copy link
Member

wangela commented Jan 25, 2023

Thanks for offering this alternative solution. We will proceed with the first pull request #417, so closing this one.

@wangela wangela closed this Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multiple avoid problems
4 participants