Skip to content

Add per-peer profit scoring#3247

Draft
t-bast wants to merge 2 commits intomasterfrom
peer-scorer
Draft

Add per-peer profit scoring#3247
t-bast wants to merge 2 commits intomasterfrom
peer-scorer

Conversation

@t-bast
Copy link
Member

@t-bast t-bast commented Feb 5, 2026

Builds on top of #3246

We improve the following events:

- `TransactionPublished` includes more details about mining fees and
  an optional liquidity purchase
- all channel events include the latest `channel_type`
- `PaymentRelayed` exposes the `relayFee` earned
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 55.40541% with 99 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.00%. Comparing base (f93d02f) to head (3473f10).
⚠️ Report is 78 commits behind head on master.

Files with missing lines Patch % Lines
...main/scala/fr/acinq/eclair/profit/PeerScorer.scala 44.69% 99 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3247      +/-   ##
==========================================
+ Coverage   86.43%   88.00%   +1.56%     
==========================================
  Files         242      219      -23     
  Lines       22607    20759    -1848     
  Branches      832      821      -11     
==========================================
- Hits        19541    18268    -1273     
+ Misses       3066     2491     -575     
Files with missing lines Coverage Δ
...ir-core/src/main/scala/fr/acinq/eclair/Setup.scala 71.97% <100.00%> (-1.40%) ⬇️
.../scala/fr/acinq/eclair/channel/ChannelEvents.scala 100.00% <100.00%> (ø)
...c/main/scala/fr/acinq/eclair/channel/Helpers.scala 92.85% <100.00%> (+0.21%) ⬆️
...acinq/eclair/channel/fsm/DualFundingHandlers.scala 88.37% <100.00%> (ø)
...la/fr/acinq/eclair/channel/fsm/ErrorHandlers.scala 82.84% <100.00%> (-0.58%) ⬇️
...inq/eclair/channel/fsm/SingleFundingHandlers.scala 78.37% <100.00%> (-1.63%) ⬇️
...cinq/eclair/channel/publish/MempoolTxMonitor.scala 87.20% <100.00%> (ø)
...main/scala/fr/acinq/eclair/db/DbEventHandler.scala 86.95% <100.00%> (-2.33%) ⬇️
...c/main/scala/fr/acinq/eclair/db/pg/PgAuditDb.scala 80.42% <100.00%> (-17.84%) ⬇️
...cala/fr/acinq/eclair/db/sqlite/SqliteAuditDb.scala 79.16% <100.00%> (-18.99%) ⬇️
... and 5 more

... and 26 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants