Skip to content

Commit 852599c

Browse files
committed
Fix clippy lint by passing errors by reference
1 parent 57be438 commit 852599c

File tree

2 files changed

+11
-7
lines changed

2 files changed

+11
-7
lines changed

src/model/preprocessing.rs

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,10 @@ where
106106
.with_n_components(n)
107107
.with_use_correlation_matrix(true),
108108
)
109-
.map_err(Self::failed_to_settings)?;
110-
let transformed = pca.transform(x).map_err(Self::failed_to_settings)?;
109+
.map_err(|err| Self::failed_to_settings(&err))?;
110+
let transformed = pca
111+
.transform(x)
112+
.map_err(|err| Self::failed_to_settings(&err))?;
111113
self.pca = Some(pca);
112114
Ok(transformed)
113115
}
@@ -123,8 +125,10 @@ where
123125

124126
fn fit_svd(&mut self, x: &InputArray, n: usize) -> Result<InputArray, SettingsError> {
125127
let svd = SVD::fit(x, SVDParameters::default().with_n_components(n))
126-
.map_err(Self::failed_to_settings)?;
127-
let transformed = svd.transform(x).map_err(Self::failed_to_settings)?;
128+
.map_err(|err| Self::failed_to_settings(&err))?;
129+
let transformed = svd
130+
.transform(x)
131+
.map_err(|err| Self::failed_to_settings(&err))?;
128132
self.svd = Some(svd);
129133
Ok(transformed)
130134
}
@@ -146,7 +150,7 @@ where
146150
ModelError::Inference(err.to_string())
147151
}
148152

149-
fn failed_to_settings(err: Failed) -> SettingsError {
153+
fn failed_to_settings(err: &Failed) -> SettingsError {
150154
SettingsError::PreProcessingFailed(err.to_string())
151155
}
152156
}

src/model/supervised.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ where
161161
self.x_train = self
162162
.preprocessor
163163
.fit_transform(raw, &sup.preprocessing)
164-
.map_err(Self::preprocessing_failed)?;
164+
.map_err(|err| Self::preprocessing_failed(&err))?;
165165

166166
for alg in <A>::all_algorithms(&self.settings) {
167167
let trained = alg.cross_validate_model(&self.x_train, &self.y_train, &self.settings)?;
@@ -209,7 +209,7 @@ where
209209
}
210210
}
211211

212-
fn preprocessing_failed(err: SettingsError) -> Failed {
212+
fn preprocessing_failed(err: &SettingsError) -> Failed {
213213
Failed::because(FailedError::ParametersError, &err.to_string())
214214
}
215215
}

0 commit comments

Comments
 (0)