Skip to content

wip: introduce py-hdwallet - #96

Closed
wolovim wants to merge 1 commit into
ApeWorX:mainfrom
wolovim:py-hdwallet-integration
Closed

wip: introduce py-hdwallet#96
wolovim wants to merge 1 commit into
ApeWorX:mainfrom
wolovim:py-hdwallet-integration

Conversation

@wolovim

@wolovim wolovim commented Apr 7, 2020

Copy link
Copy Markdown
Contributor

What does it do?

Introduces py-hdwallet for child key derivation.

(WIP: seeking feedback on items below)

Background

  • My understanding of the plan is for py-hdwallet to eventually provide all BIP32-related functionality within eth-account. This introduction of the library only goes as far as replicating existing functionality recently introduced by @fubuloubu.
  • Mnemonics are currently outside of the scope of py-hdwallet, so @fubuloubu's recent mnemonic work would go untouched.

TODO:

  • put all testing in its right place, e.g., remove or alter relevant tests in eth-account; some tests are better suited for py-hdwallet.
  • agree on hardening representation (e.g., ', h, H, etc.)
    • py-hdwallet only supports h
    • the eth-account implementation had support for ' and H

Cute Animal Picture

put a cute animal picture link inside the parentheses

@wolovim
wolovim force-pushed the py-hdwallet-integration branch from 9c78e99 to 9685bd4 Compare April 7, 2020 22:36
@fubuloubu

Copy link
Copy Markdown
Member

The ' is the standard representation IIRC. It should be supported prior to either of the other ones. Understanding it's not exactly the most legible in code, but a simple replace would solve that.

@fubuloubu

Copy link
Copy Markdown
Member

Also, a nice thing that should be introduced at some point is an alternative account type that can continuously derive new accounts using hdwallet, and encrypts/decrypts the master key instead of the account key. This would come in handy primarily for testing use (deriving multiple test accounts from a seed), but would set the stage for hardware wallet integration (which uses hd paths to obtain the account from master public key).

Comment thread setup.py
"eth-rlp>=0.1.2,<1",
"eth-utils>=1.3.0,<2",
"hexbytes>=0.1.0,<1",
"py-hdwallet>=0.1.0a1",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

BTW, we'd typically want to at least put py-hdwallet into beta before releasing this.

@fselmo

fselmo commented Feb 2, 2026

Copy link
Copy Markdown
Contributor

stale

@fselmo fselmo closed this Feb 2, 2026
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.

4 participants