search

martomi / chiadog

A watch dog providing a peace in mind that your Chia farm is running smoothly 24/7.
MIT License
457 stars 121 forks source link

Added notification when plot count changes #132

Closed jamierf closed 2 years ago

jamierf commented 3 years ago

I added a LOW level notification that fires when plot count increases. This happens on startup and whenever plot count increases.

martomi commented 3 years ago

Thanks for the proposal, this will unfortunately cause 30+ daily notifications for people plotting 3+ TB per day so I wouldn't like to merge it to the main branch.

Chardmw commented 3 years ago

Can the excess notifications be filtered as part of Issue #77.

chadmccune commented 3 years ago

I agree, even though I'm plotting about 10TB a day right now, I'd still like to have this.

On Mon, May 17, 2021 at 1:53 PM Chardmw @.***> wrote:

Can the excess notifications be filtered as part of Issue #77 https://github.com/martomi/chiadog/issues/77.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/martomi/chiadog/pull/132#issuecomment-842552917, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6RHIMR7HQSK2PH4HQSZWLTOFQY7ANCNFSM447XAFIQ .

jamierf commented 3 years ago

I can see handling it either by adding a config flag just for this event to go to notifications or a flag that enables all low level events to go to the notifications. Perhaps I'll see if I can figure that out later tonight.

andymarden commented 3 years ago

I'm using this mod (I added a pull request where I included time since last plot which is nice) - agree that it could be a config flag just for this message defaulted to false for each notification channel.

martomi commented 2 years ago

We've merged and released this feature with this implementation: https://github.com/martomi/chiadog/pull/307

Now it's possible to configure notification for increases / decreases separately.