-
Notifications
You must be signed in to change notification settings - Fork 29
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
Caravan Health Package #112
Merged
Merged
Changes from all commits
Commits
Show all changes
93 commits
Select commit
Hold shift + click to select a range
3ac2703
Initializing Caravan Health and adding privacy score
Harshil-Jani 822cf54
Utilize client methods from @caravan/client
Harshil-Jani e3989b9
Added fees score method to @caravan/health
Harshil-Jani 7737e3e
Export all privacy methods
Harshil-Jani 7421376
Add some context on the fee score functions
Harshil-Jani c3f167e
Rename to camelCase and fix wordings
Harshil-Jani 27cacf1
Setup prettier for @caravan/health and format
Harshil-Jani 29c2207
Refactor privacyScoreByTxTopology for improved readability and mainta…
Harshil-Jani 26dfd30
Add types for Transaction and UTXO for privacy.ts
Harshil-Jani f1e6a38
Changing UTXO type to Wallet UTXO which brings addresses from memory
Harshil-Jani 7f7ed1d
Fix UTXO set length comming wrong
Harshil-Jani 1f70cab
Adding Test Cases for Privacy
Harshil-Jani afe92ad
Type Improvements and code improvement
Harshil-Jani 0011b21
Better handling of exports and address functionality
Harshil-Jani 270a035
Rewrite test cases for privacy
Harshil-Jani fc589e2
Fix in privacy and it's test case
Harshil-Jani afb69a4
Completed with feescore test cases
Harshil-Jani a4c6ed7
Improving Documentation
Harshil-Jani f4959d7
Type Improvements
Harshil-Jani dd77fa3
Added Transaction[] as return type in clients fxn
Harshil-Jani ffd741c
Add dependency on @caravan-bitcoin
Harshil-Jani 49b6886
Update logic for address reuse
Harshil-Jani c39eb85
Moving Transaction type into clients
Harshil-Jani f5e6567
Performance improvement for feeRate percentile history
Harshil-Jani f71c17a
Update fees test cases
Harshil-Jani cdb7bfd
Decouple feeScore from the client
Harshil-Jani e647deb
Move tx topology as a separate function
Harshil-Jani 87c7425
Prettier Formatting
Harshil-Jani a493057
Added the changeset
Harshil-Jani d33a99d
Adding README
Harshil-Jani 68e072e
Add isSend logic for mempool and blockstream using prevouts address c…
Harshil-Jani bc977e3
Add waste metric to the feescore file
Harshil-Jani 2f2b58f
Add test case for waste metric
Harshil-Jani 64d8a18
remove @caravan/health from @caravan/clients
Harshil-Jani 6d7f289
Move interface to types.ts
Harshil-Jani 76509c9
rolling release version of @caravan-health as 1.0.0-beta
Harshil-Jani 69eb402
improving grammar and variable naming
Harshil-Jani b72beea
More descriptive changeset
Harshil-Jani 021f461
Adding call to bitcoind in bitcoind.js file
Harshil-Jani b1a2828
Descriptive Test cases added
Harshil-Jani 12b78bd
Replacing Weightage with UTXO Mass Factor
Harshil-Jani 99ca020
L to estimateLongTermFeeRate
Harshil-Jani 0a2f9ff
Improve docs
Harshil-Jani 626714b
remove precision from tests
Harshil-Jani e3a94bb
syntax improvement
Harshil-Jani 2b185fd
Introducing new class WasteMetric
Harshil-Jani 8892097
Remove deps from network client
Harshil-Jani ccad6e7
Adding PrivacyMetric class
Harshil-Jani 41bcab3
Descriptive Test cases and Object Oriented Restructure
Harshil-Jani 22b863b
Introduce Address Usage Map
Harshil-Jani a4eafb7
solve blocktime and block_time naming inconsistency
Harshil-Jani 97a8093
Cache addressUsageMap in the class
Harshil-Jani ac28f30
raise exceptions for other clients if not mempool for feerate history
Harshil-Jani 1785c8f
Adding test cases for clients methods
Harshil-Jani e66b321
Clean test cases
Harshil-Jani 7329b13
AddressUsageMap test case
Harshil-Jani e35af1d
Improved Documentation
Harshil-Jani bd7bf1e
Fixing package-lock.json
Harshil-Jani cf835e6
Storing transactions, utxos in a class
Harshil-Jani 5a5ab61
Separate out utility functions
Harshil-Jani 2ee031d
Updating documentation
Harshil-Jani 9781cc2
Adding the new UTXO dust method and updating address lookup Map
Harshil-Jani 74d656d
Make transactions and UTXO options for ad-hoc cases
Harshil-Jani 0d0ec9f
Consider Script Type for dust limits
Harshil-Jani a223d80
Fix failing tests
Harshil-Jani 1e4acb1
Update packages/caravan-health/src/waste.ts
Harshil-Jani af16010
Update packages/caravan-health/src/waste.ts
Harshil-Jani 303da1f
Fix CI errors
Harshil-Jani 4e60eb9
Update packages/caravan-health/src/waste.ts
Harshil-Jani 15f8475
Update packages/caravan-health/README.md
Harshil-Jani 1235246
Update packages/caravan-health/README.md
Harshil-Jani 6d7093b
Update packages/caravan-health/src/privacy.ts
Harshil-Jani bba82f7
Adding script type into calculateDustMetric
Harshil-Jani b4bf711
Updating test cases
Harshil-Jani 5f98fef
Code cleaning
Harshil-Jani 9b68338
Update packages/caravan-health/src/privacy.test.ts
Harshil-Jani a3032b1
Update packages/caravan-health/src/privacy.test.ts
Harshil-Jani 17de9ee
Update packages/caravan-health/src/privacy.test.ts
Harshil-Jani 8f8eff0
Update packages/caravan-health/src/privacy.test.ts
Harshil-Jani f327999
Update packages/caravan-health/src/privacy.test.ts
Harshil-Jani ac249a9
Update packages/caravan-health/src/waste.ts
Harshil-Jani 10d68d7
SpendTypeScore just takes in Input and Output
Harshil-Jani bb06e48
Adding nit for length of UTXO
Harshil-Jani d47823b
Remove unused map
Harshil-Jani 564a8ed
Dust calculations update
Harshil-Jani 2e22bf3
Update config variable names
Harshil-Jani 3a2dee1
Removed unncessary comments
Harshil-Jani afa3386
Update logic for spend type determination
Harshil-Jani 7fc3f81
Update packages/caravan-health/src/waste.ts
Harshil-Jani 5b7989d
Update packages/caravan-health/src/waste.ts
Harshil-Jani 5eec580
Update packages/caravan-health/src/privacy.ts
Harshil-Jani eb43fa4
utility.js -> spendType.js
Harshil-Jani b07b5a4
Merge branch 'main' into caravan/health
bucko13 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
--- | ||
"@caravan/bitcoin": minor | ||
"@caravan/clients": minor | ||
--- | ||
|
||
@caravan/client | ||
We are exposing a new method `getAddressTransactions` which will fetch all the transaction for a given address and format it as per needs. To facilitate the change, we had moved the interfaces in the new file `types.ts`. | ||
|
||
Another change was about getting the block fee-rate percentile history from mempool as a client. | ||
|
||
@caravan/bitcoin | ||
The new function that has the capability to detect the address type (i.e P2SH, P2PKH, P2WSH or P2TR) was added. | ||
|
||
Overall, The changes were to support the new library within caravan called @caravan/health. |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why was "UNKNOWN" necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can avoid UNKNOWN and instead show the error as it is if it occurs. It was just to keep the code consistant by adding a new default type in case of address type matching failure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I think it should just fail if it doesn't match.