Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add payment history for probability calculation in path finding #306

Open
wants to merge 45 commits into
base: main
Choose a base branch
from

Conversation

chenyukang
Copy link
Collaborator

#228 is left over because of an unnoticed merge.

@codecov-commenter
Copy link

codecov-commenter commented Nov 13, 2024

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 ⚠️
src/fiber/network.rs 77.67% 25 Missing ⚠️
src/fiber/graph.rs 92.95% 5 Missing ⚠️
Additional details and impacted files
@@            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 Coverage Δ
unittests 46.53% <81.26%> (+1.87%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@chenyukang chenyukang force-pushed the yukang-more-on-payment-session branch 2 times, most recently from 83ac324 to cc9656f Compare November 14, 2024 11:35
src/fiber/network.rs Outdated Show resolved Hide resolved
src/fiber/network.rs Show resolved Hide resolved
src/fiber/network.rs Outdated Show resolved Hide resolved
src/fiber/network.rs Outdated Show resolved Hide resolved
src/fiber/graph.rs Outdated Show resolved Hide resolved
src/fiber/graph.rs Outdated Show resolved Hide resolved
src/fiber/history.rs Outdated Show resolved Hide resolved
src/fiber/history.rs Outdated Show resolved Hide resolved
src/fiber/history.rs Outdated Show resolved Hide resolved
src/fiber/history.rs Outdated Show resolved Hide resolved
@chenyukang chenyukang force-pushed the yukang-more-on-payment-session branch 3 times, most recently from 08f8886 to 260ff2e Compare November 14, 2024 15:47
@chenyukang
Copy link
Collaborator Author

addressed all issues, thanks @quake .

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.

3 participants