-
Notifications
You must be signed in to change notification settings - Fork 14
modify import & update README.md #196
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
Changes from all commits
70ad20e
22fa4ed
b2e150c
c3a7448
4a1a2ec
9243080
aaa39c3
12d96e9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,6 +8,9 @@ prof/ | |
| .venv/ | ||
| .mypy_cache/ | ||
| __pycache__/ | ||
| .idea/ | ||
| .vscode/ | ||
| decred/setup.py | ||
| .pytest_cache | ||
| .coverage | ||
| *.c | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,6 +12,19 @@ The [`tinywallet`](./tinywallet) package contains a wallet based on the | |
| Each package may be installed from the [Python Package Index](https://pypi.org/) | ||
| using the [`pip`](https://pip.pypa.io/) command as usual. | ||
|
|
||
| ## Requirements | ||
|
|
||
| To run tinydecred on your machine you will need to set up the following. | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You don't "run tinydecred". You can run tinywallet, or use the decred package. |
||
|
|
||
| - pip install websocket_client | ||
| - pip install blake256 | ||
| - pip install base58 | ||
| - pip install PyNaCl | ||
| - pip install appdirs | ||
|
Comment on lines
+17
to
+23
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You shouldn't install this way. Regular users should just For tinywallet, you can |
||
|
|
||
| Set up and run dcrd. Ensure your rpcusername and rpcpassword is set in your dcrd config file. | ||
| Installation guide can be found [`here`](https://docs.decred.org/wallets/cli/dcrd-setup/). | ||
|
Comment on lines
+25
to
+26
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. dcrd is completely optional, and it's integration is very shallow right now. I wouldn't say much about dcrd yet. |
||
|
|
||
| ## Status | ||
|
|
||
| [](https://github.com/decred/tinydecred/actions) | ||
|
|
@@ -28,6 +41,7 @@ using the [`pip`](https://pip.pypa.io/) command as usual. | |
| [](https://github.com/decred/tinydecred/graphs/contributors) | ||
| [](./LICENSE) | ||
|
|
||
|
|
||
| ## Roadmap | ||
|
|
||
| In no particular order: | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,9 +13,9 @@ | |
| from blake256.blake256 import blake_hash | ||
| import nacl.secret | ||
|
|
||
| from decred import DecredError | ||
| from decred.util import encode | ||
| from decred.util.encode import ByteArray, unblobCheck | ||
| from decred.decred import DecredError | ||
| from decred.decred.util import encode | ||
| from decred.decred.util.encode import ByteArray, unblobCheck | ||
|
Comment on lines
-16
to
+18
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can't do this. Typically this would indicate that you are importing the package in a way that it wasn't intended. How are you using the package that requires this change?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. When I run tinywallet, without this change I was getting an importerror(unknown location), so I figured since it is absolute import, this would help. The only other fix I could figure out was the insertion of my projectpath using sys.path.insert().
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are you using the CLI command
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You'll need to Sorry this stuff isn't better documented already.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have implemented your suggestions, and I still got the error I dropped on tinydecred room..
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are you referring to the IDE error?
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, it's actually not IDE related.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Neither buck nor I are seeing this, and we use different flavors of linux I believe. I'm not great with poetry, but It looks like the virtual environment it uses isn't being set up properly on your end. |
||
|
|
||
| from . import rando | ||
| from .secp256k1.curve import PrivateKey, PublicKey, curve as Curve | ||
|
|
||
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.
For folders specific to your own environment, please use .git/info/exclude instead.