Skip to content

Clarify mining average period for hunting nodes.#151

Merged
ca333 merged 1 commit intoKomodoPlatform:s4from
smk762:patch-10
Jun 13, 2020
Merged

Clarify mining average period for hunting nodes.#151
ca333 merged 1 commit intoKomodoPlatform:s4from
smk762:patch-10

Conversation

@smk762
Copy link
Copy Markdown
Contributor

@smk762 smk762 commented Jun 13, 2020

No description provided.

@CHMEX
Copy link
Copy Markdown
Contributor

CHMEX commented Jun 13, 2020

what's the reason for
over the course of a week ?

@smk762
Copy link
Copy Markdown
Contributor Author

smk762 commented Jun 13, 2020

the previous PR mentioned a minimum 15 blocks a day on average as the threshold for low mining / hunting nodes to contribute. the timespan over which this average was to be calculated was not specified. I'm not against using a different time period, but without it being specified it would seem that "over the season" is implied, which means there is effectively no penalty for failing to achieve the threshold.

@ca333 ca333 merged commit f1931b3 into KomodoPlatform:s4 Jun 13, 2020
@TheComputerGenie
Copy link
Copy Markdown
Contributor

TheComputerGenie commented Jun 13, 2020

what's the reason for
over the course of a week ?

Because arbitrary

@smk762
Copy link
Copy Markdown
Contributor Author

smk762 commented Jun 13, 2020

what's the reason for
over the course of a week ?

Because arbitrary

No less arbitrary than 75% of top block producing node or 15 blocks a day.
A week is enough time to adjust if too low, and not too long for extended period of low production. If you had something else in mind, it should have been included in original PR.

@TheComputerGenie
Copy link
Copy Markdown
Contributor

And not once in the month that it sat there mostly ignored did this dawn on you, but had to be changed with just hrs to go before S4 starts?

@TheComputerGenie
Copy link
Copy Markdown
Contributor

If you had something else in mind, it should have been included in original PR.

I put what I had in mind, 75% of what the top node has; 15 was something @ca333 pulled out of the air, a month ago.

@smk762
Copy link
Copy Markdown
Contributor Author

smk762 commented Jun 13, 2020

earlier would have been better, but it was not identified until raised in discord today. As I am not hunting, I didnt get involved until it was raised. Not having a defined period for calculating the average renders the average pointless, so one had to be proposed. I just put what I had in mind.

@smk762 smk762 deleted the patch-10 branch June 13, 2020 17:21
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