nervosnetwork / fiber

21 stars 11 forks source link

Add payment history for probability calculation in path finding #306

Closed chenyukang closed 1 week ago

chenyukang commented 1 week ago

228 is left over because of an unnoticed merge.

codecov-commenter commented 1 week ago

Codecov Report

Attention: Patch coverage is 81.26233% with 95 lines in your changes missing coverage. Please review.

Project coverage is 46.53%. Comparing base (27d6b47) to head (ce18961).

Files with missing lines Patch % Lines
src/fiber/history.rs 77.81% 65 Missing :warning:
src/fiber/network.rs 77.67% 25 Missing :warning:
src/fiber/graph.rs 92.95% 5 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #306 +/- ## ========================================== + Coverage 44.65% 46.53% +1.87% ========================================== Files 44 45 +1 Lines 28075 28479 +404 ========================================== + Hits 12538 13253 +715 + Misses 15537 15226 -311 ``` | [Flag](https://app.codecov.io/gh/nervosnetwork/fiber/pull/306/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nervosnetwork) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/nervosnetwork/fiber/pull/306/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nervosnetwork) | `46.53% <81.26%> (+1.87%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=nervosnetwork#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

chenyukang commented 1 week ago

addressed all issues, thanks @quake .