Skip to content

Commit f7475e3

Browse files
authored
Merge pull request #2265 from MkDev11/fix/remove-duplicate-exp-functions
fix: remove duplicate safe_exp function, use exp_safe instead
2 parents 96e31af + daf6ad8 commit f7475e3

File tree

2 files changed

+1
-6
lines changed

2 files changed

+1
-6
lines changed

pallets/subtensor/src/epoch/math.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1592,8 +1592,3 @@ pub fn mat_ema_alpha(
15921592
pub fn safe_ln(value: I32F32) -> I32F32 {
15931593
ln(value).unwrap_or(I32F32::saturating_from_num(0.0))
15941594
}
1595-
1596-
/// Safe exp function, returns 0 if value is 0.
1597-
pub fn safe_exp(value: I32F32) -> I32F32 {
1598-
exp(value).unwrap_or(I32F32::saturating_from_num(0.0))
1599-
}

pallets/subtensor/src/epoch/run_epoch.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1477,7 +1477,7 @@ impl<T: Config> Pallet<T> {
14771477

14781478
// sigmoid = 1. / (1. + e^(-steepness * (combined_diff - 0.5)))
14791479
let sigmoid = one.saturating_div(
1480-
one.saturating_add(safe_exp(
1480+
one.saturating_add(exp_safe(
14811481
alpha_sigmoid_steepness
14821482
.saturating_div(I32F32::from_num(-100))
14831483
.saturating_mul(combined_diff.saturating_sub(I32F32::from_num(0.5))),

0 commit comments

Comments
 (0)