Code

Nacho/cleanup get transform util#4181

Code

Merged

SteveMacenski merged 15 commits intoros-navigation:mainros-navigation/navigation2:mainfrom nachovizzo:nacho/cleanup_get_transform_utilnachovizzo/navigation2:nacho/cleanup_get_transform_utilCopy head branch name to clipboardApr 2, 2024

+134-83

Commits

Commits on Apr 2, 2024

• remove unused header

  Show description for 829578a
  nachovizzo

  committed
  829578a

• First step. Return an optional value instead of user provided output.

  Show description for 1cbdb40
  nachovizzo

  committed
  1cbdb40

• Second step, update the consumers of this utility function

  Show description for a65880b
  nachovizzo

  committed
  a65880b

• Third step: swap source/target to make it consistent with tf lookups

  Show description for 250311d
  nachovizzo

  committed
  250311d

• transform tolerance -> transform timeout

  Show description for b1bc26f
  nachovizzo

  committed
  b1bc26f

• Last step, convert functions to template functions

  Show description for 41f8920
  nachovizzo

  committed
  41f8920

• Add nodiscard to avoid confusiong API calls

  Show description for 0b33a52
  nachovizzo

  committed
  0b33a52

• Update docs

  Show description for 0f0a75e
  nachovizzo

  committed
  0f0a75e

• Revert "transform tolerance -> transform timeout"

  Show description for d839f3b
  nachovizzo

  committed
  d839f3b

• Fix linter 🤦

  Show description for d21db92
  nachovizzo

  committed
  d21db92

• reset to main

  Show description for 2a45a68
  nachovizzo

  committed
  2a45a68

• Add 2 new utils functions to query transforms using messages

  Show description for c4c651d
  nachovizzo

  committed
  c4c651d

• Move utility function to base class to reuse implementation

  Show description for d450502
  nachovizzo

  committed
  d450502

• Merge remote-tracking branch main into nacho/cleanup_get_transform_util

  nachovizzo

  committed
  4457000

• Fix Typo

  Show description for 47ef228
  nachovizzo

  committed
  47ef228