Node basic functions implemented, major changes in most core functions#2
Open
Node basic functions implemented, major changes in most core functions#2
Conversation
…ation of node connection
…info based on what method). This makes invalid states unrepresentable
added 13 commits
June 6, 2025 17:33
…d get-transaction-pool)
…d get-transaction-pool)
…oCoin into implementing-node
Added log saving and retrieving after shutdown (on the .share/local/cleyto_coin/logs or something like that), added the config toml (on .config/cleyto_coin), new binary for creating wallets and sending transactions. Node still doesn't check the U... something to see if the sender had enough to pay the receiver. Also changed how the private and public keys are held in the wallet (from [u8] to the PKey objects.
I'm commiting now to start changing the server killing feature to a socket instead of a channel, so that I can kill it from another process and not only from another thread
Updated readme with the latest functionalities, tests will from now on always use run_server_thread instead of creating the server process. The stuff not marked with [Under development] now works, and the server runs headless unless explicitly stated --blocking or --gui
Now there's an algorithm (untested) to select the UTXOs that will compose the transactions. I should still implement a way for the user to choose it manually, introduced some utilities functions and changed some old ones. I don't remember the rest tbh
not working, trying to implement the branch and bound. I'm tired so fuck it, do it next week
Finished the implementation of the BnB algorithm for coin selection as described here https://bitcoin.stackexchange.com/questions/119919/how-does-the-branch-and-bound-coin-selection-algorithm-work. Missing implementation of non exact algorithm now, and the constants are all fucked up I think
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
The node has a HTTP server that serves as interface for connecting wallets. The Wallets have CLI commands to create them and create transactions, which now operate with UTXOs instead of just values. There is still no mechanism for communication between nodes, or deciding what is the canonical branch, but we'll get there