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.

Sign up for GitHub

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

Jump to bottom

Cleanup unused Miner Actor state #1428

Closed
aarshkshah1992 opened this issue Oct 6, 2023 · 3 comments
Closed

Cleanup unused Miner Actor state #1428

aarshkshah1992 opened this issue Oct 6, 2023 · 3 comments
Assignees
@aarshkshah1992

Comments

@aarshkshah1992
Copy link
Contributor

See miner actor section in #1274.
@aarshkshah1992 aarshkshah1992 self-assigned this Oct 6, 2023
@aarshkshah1992 aarshkshah1992 mentioned this issue Oct 6, 2023
Closed
@anorth
Copy link
Member

anorth commented Oct 8, 2023 •
edited
Loading

Except for possibly removing some constants, I don't think we should do the other things listed in that issue, at least as written. If you want to attempt them, please first write up an issue describing what and why so that we can align on what's possible and good first.

@aarshkshah1992
Copy link
Contributor Author

@anorth I think I am going to spend some time working on the Actors onboarding issues you have put together for me. A solid contextual understanding of Actors will then allow me to make changes such as these with more confidence.

@aarshkshah1992
Copy link
Contributor Author

Let's use #1274 to discuss this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aarshkshah1992 aarshkshah1992

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@anorth @aarshkshah1992