Skip to content

Commit 747a79f

Browse files
committed
reuse path names
1 parent 11518fc commit 747a79f

File tree

3 files changed

+19
-12
lines changed

3 files changed

+19
-12
lines changed

tools/twix/src/panels/automatic_camera_calibration_export.rs

+3-2
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,9 @@ use parameters::directory::Scope;
1111

1212
use crate::{log_error::LogError, nao::Nao, panel::Panel, value_buffer::BufferHandle};
1313

14-
const TOP_CAMERA_EXTRINSICS_PATH: &str = "camera_matrix_parameters.vision_top.extrinsic_rotations";
15-
const BOTTOM_CAMERA_EXTRINSICS_PATH: &str =
14+
pub const TOP_CAMERA_EXTRINSICS_PATH: &str =
15+
"camera_matrix_parameters.vision_top.extrinsic_rotations";
16+
pub const BOTTOM_CAMERA_EXTRINSICS_PATH: &str =
1617
"camera_matrix_parameters.vision_bottom.extrinsic_rotations";
1718
pub struct AutomaticCameraCalibrationExportPanel {
1819
nao: Arc<Nao>,

tools/twix/src/panels/manual_camera_calibration.rs

+12-9
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,13 @@ use nalgebra::Vector3;
77
use parameters::directory::Scope;
88
use serde_json::Value;
99

10-
use crate::{log_error::LogError, nao::Nao, panel::Panel, value_buffer::BufferHandle};
10+
use crate::{
11+
log_error::LogError,
12+
nao::Nao,
13+
panel::Panel,
14+
panels::{BOTTOM_CAMERA_EXTRINSICS_PATH, TOP_CAMERA_EXTRINSICS_PATH},
15+
value_buffer::BufferHandle,
16+
};
1117

1218
pub struct ManualCalibrationPanel {
1319
nao: Arc<Nao>,
@@ -19,12 +25,9 @@ impl Panel for ManualCalibrationPanel {
1925
const NAME: &'static str = "Manual Calibration";
2026

2127
fn new(nao: Arc<Nao>, _value: Option<&Value>) -> Self {
22-
let top_camera = nao.subscribe_value(
23-
"parameters.camera_matrix_parameters.vision_top.extrinsic_rotations".to_string(),
24-
);
25-
let bottom_camera = nao.subscribe_value(
26-
"parameters.camera_matrix_parameters.vision_bottom.extrinsic_rotations".to_string(),
27-
);
28+
let top_camera = nao.subscribe_value(format!("parameters.{TOP_CAMERA_EXTRINSICS_PATH}"));
29+
let bottom_camera =
30+
nao.subscribe_value(format!("parameters.{BOTTOM_CAMERA_EXTRINSICS_PATH}"));
2831

2932
Self {
3033
nao,
@@ -44,7 +47,7 @@ impl Widget for &mut ManualCalibrationPanel {
4447
"Top Camera",
4548
value,
4649
&self.nao,
47-
"camera_matrix_parameters.vision_top.extrinsic_rotations",
50+
TOP_CAMERA_EXTRINSICS_PATH,
4851
);
4952
}
5053
ui.separator();
@@ -54,7 +57,7 @@ impl Widget for &mut ManualCalibrationPanel {
5457
"Bottom Camera",
5558
value,
5659
&self.nao,
57-
"camera_matrix_parameters.vision_bottom.extrinsic_rotations",
60+
BOTTOM_CAMERA_EXTRINSICS_PATH,
5861
);
5962
}
6063
})

tools/twix/src/panels/mod.rs

+4-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,10 @@ mod remote;
1414
mod text;
1515
mod vision_tuner;
1616

17-
pub use automatic_camera_calibration_export::AutomaticCameraCalibrationExportPanel;
17+
pub use automatic_camera_calibration_export::{
18+
AutomaticCameraCalibrationExportPanel, BOTTOM_CAMERA_EXTRINSICS_PATH,
19+
TOP_CAMERA_EXTRINSICS_PATH,
20+
};
1821
pub use ball_candidates::BallCandidatePanel;
1922
pub use behavior_simulator::BehaviorSimulatorPanel;
2023
pub use enum_plot::EnumPlotPanel;

0 commit comments

Comments
 (0)