Wrong abs used in TranslationFactor

[ProfFan]

/home/fan/src/gtsam/gtsam/sfm/TranslationFactor.h:137:30: warning: using integer absolute value function 'abs' when argument is of floating point type [-Wabsolute-value]
  137 |     const double abs_scale = abs(scale[0]);
      |                              ^
/home/fan/src/gtsam/gtsam/sfm/TranslationFactor.h:137:30: note: use function 'std::abs' instead
  137 |     const double abs_scale = abs(scale[0]);
      |                              ^~~
      |                              std::abs

[varunagrawal]

Is this a false negative? abs is overloaded and unless we're using the C version, I guess the main thing is to just add the std::