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

Monero Cryptonight variants, and add one for v7 #1072

Closed
wants to merge 1 commit into from

Conversation

moneromooo
Copy link

Please make sure your PR is against dev branch. Merging PRs directly into master branch would interfere with our workflow.

@moneromooo
Copy link
Author

@psychocrypt
Copy link
Collaborator

Thank you for providing the patch to support the upcoming POW change.
Never the less I will point to monero-project/monero#3253 (comment).

@moneromooo Is there a test net running with the new pow and is there a pool available?

@fireice-uk
Copy link
Owner

I will have a mock pool to test performance (not correctness) up tomorrow.

@fireice-uk
Copy link
Owner

With regards to merging, I'm against it as-is. When the core team actually settles on an algorithm we can decide.

@fireice-uk
Copy link
Owner

I will close this PR as the change looks half-cocked and doesn't match the algorithm under consideration now.

@fireice-uk fireice-uk closed this Feb 21, 2018
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.

3 participants