Skip to content

Commit

Permalink
Remove prints
Browse files Browse the repository at this point in the history
  • Loading branch information
Matt M committed Sep 17, 2024
1 parent e5cf605 commit 9ed6069
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
6 changes: 1 addition & 5 deletions photon-lib/src/main/native/cpp/photon/PhotonCamera.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,6 @@ PhotonCamera::PhotonCamera(nt::NetworkTableInstance instance,
topicNameSubscriber(instance, PHOTON_PREFIX, {.topicsOnly = true}),
path(rootTable->GetPath()),
cameraName(cameraName) {
fmt::println("Created photoncamera for {}", cameraName);
HAL_Report(HALUsageReporting::kResourceType_PhotonCamera, InstanceCount);
InstanceCount++;
}
Expand Down Expand Up @@ -187,10 +186,7 @@ void PhotonCamera::TakeOutputSnapshot() {
outputSaveImgEntry.Set(outputSaveImgSubscriber.Get() + 1);
}

bool PhotonCamera::GetDriverMode() const {
fmt::println("driver mode? {}", driverModeSubscriber.Get());
return driverModeSubscriber.Get();
}
bool PhotonCamera::GetDriverMode() const { return driverModeSubscriber.Get(); }

void PhotonCamera::SetPipelineIndex(int index) { pipelineIndexPub.Set(index); }

Expand Down
6 changes: 3 additions & 3 deletions photon-lib/src/test/native/cpp/VisionSystemSimTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -443,9 +443,9 @@ TEST_F(VisionSystemSimTest, TestPoseEstimation) {
camEigen, distEigen, targets, layout, photon::kAprilTag16h5);
ASSERT_TRUE(results);
frc::Pose3d pose = frc::Pose3d{} + results->best;
ASSERT_NEAR(5, pose.X().to<double>(), 0.01);
ASSERT_NEAR(1, pose.Y().to<double>(), 0.01);
ASSERT_NEAR(0, pose.Z().to<double>(), 0.01);
ASSERT_NEAR(5, pose.X().to<double>(), 0.02);
ASSERT_NEAR(1, pose.Y().to<double>(), 0.02);
ASSERT_NEAR(0, pose.Z().to<double>(), 0.02);
ASSERT_NEAR(units::degree_t{5}.convert<units::radians>().to<double>(),
pose.Rotation().Z().to<double>(), 0.01);

Expand Down

0 comments on commit 9ed6069

Please sign in to comment.