lamassu/lamassu-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: use getSolution

Browse source
This commit is contained in:
siiky 2023-11-24 12:11:02 +00:00
parent 1b30f43747
commit 8e33bc527d
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
lib/bill-math.js
View file

@ -1,6 +1,7 @@
const _ = require('lodash/fp')
const sumService = require('@haensl/subset-sum')
const BN = require('./bn')
const logger = require('./logger')
const cc = require('./coin-change')
@ -152,7 +153,7 @@ const solutionToOriginalUnits = (solution, units) => {
function makeChange(outCassettes, amount) {
const ss_solution = getSolution_old(outCassettes, amount, BILL_LIST_MODES.VALUE_ROUND_ROBIN)
const cc_solution = cc.solve(cc.model(outCassettes), amount.toNumber())
const cc_solution = getSolution(outCassettes, amount)
if (!!ss_solution === !!cc_solution) {
logger.error(new Error(`subset-sum and coin-change don't agree on solvability -- subset-sum:${!!ss_solution} coin-change:${!!cc_solution}`))