Fix C4305 double truncated to float warnings (#666) (#668)

(cherry picked from commit 433d1cd)

Signed-off-by: Jorge J. Perez <[email protected]>
Co-authored-by: Jorge J. Perez <[email protected]>

Author: mergify[bot]

graphics/src/AssimpLoader_TEST.cc

@@ -684,7 +684,7 @@ TEST_F(AssimpLoader, LoadGlTF2BoxTransmission)
   const common::MaterialPtr mat = mesh->MaterialByIndex(0u);
   ASSERT_TRUE(mat.get());
   // transmission currently modeled as transparency
-  EXPECT_FLOAT_EQ(0.1, mat->Transparency());
+  EXPECT_FLOAT_EQ(0.1f, mat->Transparency());
   delete mesh;
 #endif
 }
@@ -790,8 +790,8 @@ TEST_F(AssimpLoader, LoadGlbPbrAsset)
   EXPECT_NE(pbr->MetalnessMapData(), nullptr);
   EXPECT_NE(pbr->RoughnessMapData(), nullptr);
   // Check pixel values to test metallicroughness texture splitting
-  EXPECT_FLOAT_EQ(pbr->MetalnessMapData()->Pixel(256, 256).R(), 0.0);
-  EXPECT_FLOAT_EQ(pbr->RoughnessMapData()->Pixel(256, 256).R(), 124.0 / 255.0);
+  EXPECT_FLOAT_EQ(pbr->MetalnessMapData()->Pixel(256, 256).R(), 0.0f);
+  EXPECT_FLOAT_EQ(pbr->RoughnessMapData()->Pixel(256, 256).R(), 124.0f / 255.0f);
 
   // Bug in assimp 5.0.x that doesn't parse coordinate sets properly
   // \todo(iche033) Lightmaps are disabled for glb meshes