Skip to content

Applying blocks API#896

Closed
danielabrozzoni wants to merge 2 commits intobitcoindevkit:masterfrom
danielabrozzoni:applying_blocks
Closed

Applying blocks API#896
danielabrozzoni wants to merge 2 commits intobitcoindevkit:masterfrom
danielabrozzoni:applying_blocks

Conversation

@danielabrozzoni
Copy link
Copy Markdown
Member

@danielabrozzoni danielabrozzoni commented Mar 16, 2023

Closes #903

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

@danielabrozzoni danielabrozzoni force-pushed the applying_blocks branch 2 times, most recently from 462d2e5 to 88386ef Compare March 16, 2023 16:38
@danielabrozzoni danielabrozzoni changed the title Applying blocks Applying blocks API Mar 16, 2023
- ChainGraph::determine_relevant_changeset
- KeychainTracker::determine_relevant_changeset,
  KeychainTracker::determine_additions,
  KeychainTracker::apply_update_relevant,
- TxGraph::determine_relevant_additions
@notmandatory
Copy link
Copy Markdown
Member

@danielabrozzoni does this PR need to be rebase/worked now that that #976 is merge?

@danielabrozzoni
Copy link
Copy Markdown
Member Author

I'm really not sure, @evanlinjin is this PR still needed?

@evanlinjin
Copy link
Copy Markdown
Member

I'm really not sure, @evanlinjin is this PR still needed?

It definitely needs further discussion. It depends if everyone is happy with my approach in #1041, or do we feel that having an apply_blocks method is strictly better?

@LLFourn
Copy link
Copy Markdown
Collaborator

LLFourn commented Aug 9, 2023

I'm really not sure, @evanlinjin is this PR still needed?

It definitely needs further discussion. It depends if everyone is happy with my approach in #1041, or do we feel that having an apply_blocks method is strictly better?

I think having apply_block is a better approach after reviewing #1041.

@danielabrozzoni
Copy link
Copy Markdown
Member Author

Closing as the approach changed radically anyways

@danielabrozzoni danielabrozzoni deleted the applying_blocks branch October 11, 2023 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Applying blocks APIs

4 participants