Code

Implement lowest fee metric correctly#13

Code

Merged

evanlinjin merged 7 commits intobitcoindevkit:masterbitcoindevkit/coin-select:masterfrom LLFourn:fix-lowest-feeLLFourn/coin-select:fix-lowest-feeCopy head branch name to clipboardJan 15, 2024

+303-221

Commits

Commits on Dec 22, 2023

• Rethink is_target_met

  Show description for 0c66696
  LLFourn

  committed
  0c66696

• Make lowest fee test fail by implementing score correctly

  LLFourn

  committed
  0aef6ff

• Fix lowest_fee metric

  LLFourn

  committed
  e30246d

• Score branches before adding children

  Show description for 17cc8f2
  LLFourn

  committed
  17cc8f2

• Write lowest fee test that hits important branch

  Show description for 7360052
  LLFourn

  committed
  7360052

• Use ChangePolicy::min_value in lowest fee tests

  Show description for 9e1cecd
  LLFourn

  committed
  9e1cecd

Commits on Jan 15, 2024

• docs: fix typos and use better wording

  evanlinjin

  committed
  6ae0fdf