Martin Mihaylov

Results 27 comments of Martin Mihaylov

The packages are available here: https://github.com/martomi/chiadog/pkgs/container/chiadog In https://github.com/martomi/chiadog/pull/337 I've added a LABEL to the Dockerfile which should hopefully make the link to the repository. Then I expect the Packages section...

It's still not appearing on the main page of the repo. I'm not sure why. If anyone figures it out, let me know.

You'll probably want to remove it from here as well https://github.com/martomi/chiadog/blob/da1a904466e6df25b930d58cefb70eca6dce4a3a/src/notifier/keep_alive_monitor.py#L30 otherwise the keep-alive is still going to notify you

Install script checks `python3 --version` and not `python --version`. You probably have older version under `python3`

These are very valid points, thanks for the feedback! We should add explicit check for non-Debian based distributions and exit if we decide to keep the install script. And also...

Interesting idea. How do you see the benefits/disadvantages compared to this guide for monitoring remote harvesters from a central place? https://github.com/martomi/chiadog/wiki/Monitoring-Multiple-Harvesters ?

Seems related to https://github.com/martomi/chiadog/issues/327 we should catch that exception, log it, and retry without killing the thread.

Yep, I've [added handling](https://github.com/martomi/chiadog/pull/28/commits/fcfbc76ab0077e0453a86d3a6279d5a25bebaef3) for the uncaught exceptions yesterday to `dev`. Queueing and retry is yet to be implemented though. If someone want to implement queueing, I'd suggest doing so...

The tricky part will be to persist the data reliably even if the program doesn't exit nominally. What if the user closes the terminal window that runs `chiadog`? It seems...

Good insights! I'd suggest to do it a bit simpler and only consider challenges with at least 1 eligible plot for the statistics. The data about 0 eligible plots doesn't...