fabianonline / OctoPrint-Telegram

Plugin for octoprint to send status messages and receive commands via Telegram messenger.
312 stars 114 forks source link

no notification via SD card print #160

Open MLWORKX opened 5 years ago

MLWORKX commented 5 years ago

Hi,

When printing via OctoPrint I do get start + end notifications plus time and layer hight driven notifications: great!

When saving the files on the sd card I only do get start and end notifications as the printer does not report the actual height back to Octoprint (see picture!) http://666kb.com/i/e1oue1xhtlrr84dr3.jpg

...is there ANY way of telling this Plugin to message me at least every n minutes?

fabianonline commented 5 years ago

At the moment not. The time based notification code is only triggered on z changes, so it is more a "notify on the first z change at least x minutes after the last notification". But I think having some kind of background thread sending a notification every x minutes would be possible and useful.

GilDev commented 5 years ago

This would be nice indeed, I like printing from the SD card because of the reliability.

kanocz commented 5 years ago

@MLWORKX I think it's possible to implement in separate plugin I'm working now on https://github.com/kanocz/octopi_eta_override - receiving Prusa's updates about ETA and sets it... but I can combine this with query of current position and try to emit "z-change" event with possition every minute... so if z is changed enough telegram plugin will send notification

(edit) P.S.: just implemented this and looks like works without any issues

kanocz commented 5 years ago

@MLWORKX I think this issue can be closed now

MLWORKX commented 5 years ago

@MLWORKX I think this issue can be closed now

Thank you very much for your efford - amazing!

I'll test it till end of week. I feel it should be a part of Octoprint itself without a plugin therefore I'm not sure if it should be closed even if it works like a charm (?).

kanocz commented 5 years ago

@MLWORKX you know... maybe you're right :) my solution works only for Prusa MK3(S) printers as their slicer has extended time estimaton and notfication system... just forgot that any other printers exist :(