Skip to content

Conversation

LefterisJP
Copy link
Member

Fix #9547

For now simply using a python only implementation by Pieter Wuille from bitcoin's repository:
https://github.com/bitcoin/bitcoin/blob/ad3e9e1f214d739e098c6ebbd300da5df1026a44/test/functional/test_framework/ripemd160.py

Copy link

codecov bot commented Mar 1, 2025

Codecov Report

Attention: Patch coverage is 82.35294% with 9 lines in your changes missing coverage. Please review.

Project coverage is 55.91%. Comparing base (1c415f5) to head (a3f2c24).
Report is 373 commits behind head on bugfixes.

Files with missing lines Patch % Lines
rotkehlchen/chain/bitcoin/utils.py 44.44% 4 Missing and 1 partial ⚠️
rotkehlchen/chain/bitcoin/ripemd160.py 90.00% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           bugfixes    #9550      +/-   ##
============================================
+ Coverage     54.62%   55.91%   +1.29%     
============================================
  Files          1908     1888      -20     
  Lines        141491   138700    -2791     
  Branches      17507    13739    -3768     
============================================
+ Hits          77292    77558     +266     
+ Misses        61340    58341    -2999     
+ Partials       2859     2801      -58     
Flag Coverage Δ
backend 80.97% <82.35%> (+0.04%) ⬆️

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.

Copy link
Member

@yabirgb yabirgb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@LefterisJP
Copy link
Member Author

okay so the tests passed. Will revert the CI changes so all tests run again.

@LefterisJP
Copy link
Member Author

ah this should be against bugfixes

@LefterisJP LefterisJP changed the base branch from develop to bugfixes March 1, 2025 21:54
@LefterisJP LefterisJP merged commit ab4f46a into rotki:bugfixes Mar 1, 2025
17 checks passed
@LefterisJP LefterisJP deleted the fix_9547 branch March 1, 2025 22:08
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.

In 1.38 BTC query failed for ripemd160
2 participants