Skip to content
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

fix: get sha3 mining working #244

Merged
merged 1 commit into from
Nov 21, 2023
Merged

fix: get sha3 mining working #244

merged 1 commit into from
Nov 21, 2023

Conversation

CjS77
Copy link
Contributor

@CjS77 CjS77 commented Nov 21, 2023

  • Main fix was getting dependency management right.

There isn't true automatic dep management. The Session struct has some boolean methods like is_wallet_active() that masks real dependencies. We should really have proper dependencies that don't rely on this hack.

  • Fixing configuration in config.toml

Plenty of niggly settings (and many hours) to get this right. Sigh.

  • Automatically configuring the custom base node based on its pubkey.
  • Made node and wallet interactive so that docker attach works. This made debugging waaaaay easier.

Added some other logs which could come out, or become trace in future.

* Main fix was getting dependency management right.

There isn't _true_ automatic dep management. The `Session` struct has
some boolean methods like `is_wallet_active()` that masks real
dependencies. We should really have proper dependencies that don't rely
on this hack.

* Fixing configuration in config.toml

Plenty of niggly settings (and many hours) to get this right. Sigh.

* Automatically configuring the custom base node based on its pubkey.
* Made node and wallet interactive so that `docker attach` works. This
  made debugging waaaaay easier.

Added some other logs which could come out, or become trace in future.
@CjS77 CjS77 force-pushed the launch_containers branch from f7f6eb8 to 055a623 Compare November 21, 2023 19:57
@CjS77 CjS77 merged commit f39582a into main Nov 21, 2023
9 checks passed
@CjS77 CjS77 deleted the launch_containers branch November 21, 2023 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants