open.source/manifold
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

update metrics: fix divide by zero, elasticity NaN bug

Browse Source
This commit is contained in:
mantikoros 2022-10-08 12:16:38 -05:00
parent d00ea65279
commit e1636d0f13
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
common/calculate-metrics.ts
View File

@ -97,7 +97,11 @@ export const computeBinaryCpmmElasticity = (
)
const resultNo = getCpmmProbability(poolN, pN)
return resultYes - resultNo
// handle AMM overflow
const safeYes = Number.isFinite(resultYes) ? resultYes : 1
const safeNo = Number.isFinite(resultNo) ? resultNo : 0
return safeYes - safeNo
}
export const computeDpmElasticity = (

2
functions/src/update-metrics.ts
View File

@ -169,7 +169,7 @@ export async function updateMetricsCore() {
return 0
}
const contractRatio =
contract.flaggedByUsernames.length / (contract.uniqueBettorCount ?? 1)
contract.flaggedByUsernames.length / (contract.uniqueBettorCount || 1)
return contractRatio
})