[RSDK-10768] Use the expected name for single input tensors (viamrobotics#439)

Author: acmorrow

README.md

@@ -37,7 +37,7 @@ docker build --build-arg BASE_TAG=base/bullseye --build-arg GIT_TAG=[...] -f etc
 ```
 
 This will use `base/bullseye` as a base to build the SDK version provided in `GIT_TAG`,
-which should be a tagged release version. The SDK will be cloned from 
+which should be a tagged release version. The SDK will be cloned from
 https://github.com/viamrobotics/viam-cpp-sdk/. This is the recommended approach for
 C++ module development, which should generally be done against a tagged release.
 
@@ -47,22 +47,22 @@ docker build --build-arg BASE_TAG=base/bullseye --build-arg REPO_SETUP=copy -f e
 ```
 
 Note the use of the build argument `REPO_SETUP=copy`, which adds a Docker instruction
-to copy the SDK repo from the current working directory, rather than cloning from 
+to copy the SDK repo from the current working directory, rather than cloning from
 GitHub. This approach may make more sense for developing on the SDK itself, or if
 your C++ SDK development relies on a localversion of the SDK.
 
 The examples above illustrated the use of several `--build-arg` arguments, namely
-`BASE_TAG`, `GIT_TAG`, and `REPO_SETUP`. Please see 
+`BASE_TAG`, `GIT_TAG`, and `REPO_SETUP`. Please see
 [Dockerfile.sdk-build](etc/docker/Dockerfile.sdk-build) for a complete account of
 all build arguments and their defaults.
 
 ## Building Documentation Locally for Testing
 The C++ sdk uses [Doxygen](https://www.doxygen.nl/) to generate documentation.
 An automated workflow will generate and update our documentation on each merge,
-and publish it to [cpp.viam.dev](https://cpp.viam.dev). 
+and publish it to [cpp.viam.dev](https://cpp.viam.dev).
 
 Generating documentation locally to observe changes while developing with the
-C++ SDK is simple. 
+C++ SDK is simple.
 First, make sure doxygen is installed, e.g.,
 ```
 (on mac) brew install doxygen
@@ -91,10 +91,10 @@ quickly as possible.
 ## A note on logging
 
 Users should only interact with logging via the macros, classes, and functions in
-[`viam/sdk/log/logging.hpp`](src/viam/sdk/log/logging.hpp). Logging is 
+[`viam/sdk/log/logging.hpp`](src/viam/sdk/log/logging.hpp). Logging is
 implemented using Boost.Log, but this is an implementation detail subject
 to change without warning. In particular, using Boost.Log macros such as
-`BOOST_LOG_TRIVIAL` or `BOOST_LOG_SEV` is undefined behavior which will likely 
+`BOOST_LOG_TRIVIAL` or `BOOST_LOG_SEV` is undefined behavior which will likely
 fail to output log messages.
 
 ## License

src/viam/examples/modules/complex/gizmo/api.cpp

@@ -24,12 +24,12 @@ API API::traits<Gizmo>::api() {
     return {"viam", "component", "gizmo"};
 }
 
-Gizmo::Gizmo(std::string name) : Component(std::move(name)){};
+Gizmo::Gizmo(std::string name) : Component(std::move(name)) {}
 
 /* Gizmo server methods */
 
 GizmoServer::GizmoServer(std::shared_ptr<ResourceManager> manager)
-    : ResourceServer(std::move(manager)){};
+    : ResourceServer(std::move(manager)) {}
 
 grpc::Status GizmoServer::DoOne(grpc::ServerContext* context,
                                 const DoOneRequest* request,
@@ -170,7 +170,7 @@ grpc::Status GizmoServer::DoTwo(::grpc::ServerContext* context,
 /* Gizmo client methods */
 
 GizmoClient::GizmoClient(std::string name, std::shared_ptr<grpc::Channel> channel)
-    : Gizmo(std::move(name)), stub_(GizmoService::NewStub(channel)), channel_(std::move(channel)){};
+    : Gizmo(std::move(name)), stub_(GizmoService::NewStub(channel)), channel_(std::move(channel)) {}
 
 bool GizmoClient::do_one(std::string arg1) {
     return make_client_helper(this, *stub_, &StubType::DoOne)

src/viam/examples/modules/complex/gizmo/impl.hpp

@@ -14,10 +14,10 @@ using namespace viam::sdk;
 // `validate` method that checks config validity.
 class MyGizmo : public Gizmo, public Reconfigurable {
    public:
-    MyGizmo(std::string name, std::string arg1) : Gizmo(std::move(name)), arg1_(std::move(arg1)){};
+    MyGizmo(std::string name, std::string arg1) : Gizmo(std::move(name)), arg1_(std::move(arg1)) {}
     MyGizmo(const Dependencies& deps, const ResourceConfig& cfg) : Gizmo(cfg.name()) {
         this->reconfigure(deps, cfg);
-    };
+    }
     void reconfigure(const Dependencies& deps, const ResourceConfig& cfg) override;
     static std::vector<std::string> validate(ResourceConfig cfg);
 

src/viam/examples/modules/complex/summation/api.cpp

@@ -24,12 +24,12 @@ API API::traits<Summation>::api() {
     return {"viam", "service", "summation"};
 }
 
-Summation::Summation(std::string name) : Service(std::move(name)){};
+Summation::Summation(std::string name) : Service(std::move(name)) {}
 
 /* Summation server methods */
 
 SummationServer::SummationServer(std::shared_ptr<ResourceManager> manager)
-    : ResourceServer(std::move(manager)){};
+    : ResourceServer(std::move(manager)) {}
 
 grpc::Status SummationServer::Sum(grpc::ServerContext* context,
                                   const SumRequest* request,
@@ -58,7 +58,7 @@ grpc::Status SummationServer::Sum(grpc::ServerContext* context,
 SummationClient::SummationClient(std::string name, std::shared_ptr<grpc::Channel> channel)
     : Summation(std::move(name)),
       stub_(SummationService::NewStub(channel)),
-      channel_(std::move(channel)){};
+      channel_(std::move(channel)) {}
 
 double SummationClient::sum(std::vector<double> numbers) {
     return make_client_helper(this, *stub_, &StubType::Sum)

src/viam/examples/modules/complex/summation/impl.hpp

@@ -13,10 +13,10 @@ using namespace viam::sdk;
 class MySummation : public Summation, public Reconfigurable {
    public:
     MySummation(std::string name, bool subtract)
-        : Summation(std::move(name)), subtract_(subtract){};
+        : Summation(std::move(name)), subtract_(subtract) {}
     MySummation(const Dependencies& deps, const ResourceConfig& cfg) : Summation(cfg.name()) {
         this->reconfigure(deps, cfg);
-    };
+    }
     void reconfigure(const Dependencies& deps, const ResourceConfig& cfg) override;
     static std::vector<std::string> validate(ResourceConfig cfg);
 

src/viam/sdk/components/base.cpp

@@ -24,7 +24,7 @@ bool operator==(const Base::properties& lhs, const Base::properties& rhs) {
            lhs.turning_radius_meters == rhs.turning_radius_meters;
 }
 
-Base::Base(std::string name) : Component(std::move(name)){};
+Base::Base(std::string name) : Component(std::move(name)) {}
 
 }  // namespace sdk
 }  // namespace viam

src/viam/sdk/components/board.cpp

@@ -16,7 +16,7 @@ API API::traits<Board>::api() {
     return {kRDK, kComponent, "board"};
 }
 
-Board::Board(std::string name) : Component(std::move(name)){};
+Board::Board(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Board::status& lhs, const Board::status& rhs) {
     return (lhs.analog_reader_values == rhs.analog_reader_values &&

src/viam/sdk/components/camera.cpp

@@ -130,7 +130,7 @@ std::string Camera::normalize_mime_type(const std::string& str) {
     return mime_type;
 }
 
-Camera::Camera(std::string name) : Component(std::move(name)){};
+Camera::Camera(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Camera::point_cloud& lhs, const Camera::point_cloud& rhs) {
     return lhs.mime_type == rhs.mime_type && lhs.pc == rhs.pc;

src/viam/sdk/components/encoder.cpp

@@ -15,7 +15,7 @@ API API::traits<Encoder>::api() {
     return {kRDK, kComponent, "encoder"};
 }
 
-Encoder::Encoder(std::string name) : Component(std::move(name)){};
+Encoder::Encoder(std::string name) : Component(std::move(name)) {}
 
 bool operator==(const Encoder::position& lhs, const Encoder::position& rhs) {
     return (lhs.value == rhs.value && lhs.type == rhs.type);

src/viam/sdk/components/generic.cpp

@@ -11,7 +11,7 @@ API API::traits<GenericComponent>::api() {
     return {kRDK, kComponent, "generic"};
 }
 
-GenericComponent::GenericComponent(std::string name) : Component(std::move(name)){};
+GenericComponent::GenericComponent(std::string name) : Component(std::move(name)) {}
 
 }  // namespace sdk
 }  // namespace viam