ValuedMammal

A few thoughts: - move Balance into tx_graph.rs, as suggested already - re-export `tx_graph::Balance` in chain/src/lib.rs - re-export `bdk_chain::Balance` in `bdk::wallet` mod (as usual) Undecided what to do with `tx_data_traits`...

@startup-dreamer A single pr for moving Balance, but I would like to get the opinion of the maintainers before formulating anything concrete.

If I understand, you're using range in a couple different ways: in keychain/txout_index to range over keychains, and in spk_txout_index to range over derivation indexes - with `map_to_inner_bounds` having some...

I'd be interested in picking this up if no one has yet started. I may have questions. I can see some working examples such as #1055 and also LLFourn/bdk_core_staging#153. If...

Ideally, BDK and nakamoto should depend on the same version of rust bitcoin, or else we run into type mismatch errors that are cumbersome to try to work around. If...

Feel free to leave suggestions on #1221 as well, where the idea is to come up with more idiomatic names

I'm in favor of this suggestion https://github.com/bitcoindevkit/bdk/pull/1225#discussion_r1403327497. To quote @danielabrozzoni: ```rust let past_gap_limit = if let Some(i) = last_active_index { last_index >= i.saturating_add(stop_gap as u32) } else { // If...

Now that #1351 is open, I made LagginTimes/bdk#1 to test stop_gap in `ElectrumExt::full_scan`

possibly related #1419

BDK can possibly expose a method, probably from rust-miniscript, that just handles the finalizer role for a signed psbt. That said, I would expect a method called `sign` to fail...