taxilian / OctoPrint-Twilio

Octoprint plugin for print completion notifications using Twilio (for free w/ trial account)
GNU Affero General Public License v3.0
5 stars 9 forks source link

How to debug problem? #10

Closed mark-hahn closed 6 years ago

mark-hahn commented 6 years ago

I have two identical raspi setups, one for each of two printers. This plugin works on one raspi and not the other even though I copied the working settings to the other printer. How can I chase this problem?

P.S. A test button would be most welcome.

taxilian commented 6 years ago

Best way to find issues is usually to look at the octoprint logs

On Wed, Jun 6, 2018 at 2:04 PM, Mark Hahn notifications@github.com wrote:

I have two identical raspi setups, one for each of two printers. This plugin works on one raspi and not the other even though I copied the working settings to the other printer. How can I chase this problem?

P.S. A test button would be most welcome.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/taxilian/OctoPrint-Twilio/issues/10, or mute the thread https://github.com/notifications/unsubscribe-auth/AANsTkiBtxiYSv_C-1g7ntVInyKKHq2Hks5t6DXVgaJpZM4UdR_n .

mark-hahn commented 6 years ago

Here is the log entry for the problem. Can you help me decode this?

2018-06-06 04:22:55,065 - octoprint.plugin - ERROR - Error while calling plugin smsnotifier
Traceback (most recent call last):
  File "/home/pi/oprint/local/lib/python2.7/site-packages/octoprint/plugin/__init__.py", line 225, in call_plugin
    result = getattr(plugin, method)(*args, **kwargs)
  File "/home/pi/oprint/local/lib/python2.7/site-packages/octoprint_smsnotifier/__init__.py", line 67, in on_event
    self._process_snapshot(snapshot_path)
  File "/home/pi/oprint/local/lib/python2.7/site-packages/octoprint_smsnotifier/__init__.py", line 155, in _process_snapshot
    ffmpeg_command += ["-vf", sarge.shell_quote(",".join(rotate_params)), snapshot_path]
NameError: global name 'sarge' is not defined
mark-hahn commented 6 years ago

FYI: sudo pip install sarge didn't help.

2018-06-06 21:09:35,296 - octoprint.plugin - ERROR - Error while calling plugin smsnotifier
Traceback (most recent call last):
  File "/home/pi/oprint/local/lib/python2.7/site-packages/octoprint/plugin/__init__.py", line 225, in call_plugin
    result = getattr(plugin, method)(*args, **kwargs)
  File "/home/pi/oprint/local/lib/python2.7/site-packages/octoprint_smsnotifier/__init__.py", line 67, in on_event
    self._process_snapshot(snapshot_path)
  File "/home/pi/oprint/local/lib/python2.7/site-packages/octoprint_smsnotifier/__init__.py", line 155, in _process_snapshot
    ffmpeg_command += ["-vf", sarge.shell_quote(",".join(rotate_params)), snapshot_path]
NameError: global name 'sarge' is not defined
mark-hahn commented 6 years ago

Turning off the image feature fixed it but I would really like to have images.

tedder commented 6 years ago

Did you install it through Octoprint? (eg not manually)

shadycuz commented 6 years ago

Mark I added the SMS feature, I'm traveling but I can look at it this weekend. It must be a package dependency issue.

On Wed, Jun 6, 2018, 2:26 PM Mark Hahn notifications@github.com wrote:

Turning off the image feature fixed it but I would really like to have images.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/taxilian/OctoPrint-Twilio/issues/10#issuecomment-395219354, or mute the thread https://github.com/notifications/unsubscribe-auth/AQTJt5RBD8hPls4eiakxEo-kqfVzotC8ks5t6EkigaJpZM4UdR_n .

tedder commented 6 years ago

yeah, looks like sarge only shows up in the .py, not as a dependency.

mark-hahn commented 6 years ago

That was quick. Thanks to all. I assume I'll get a notification in octoprint when a new version is out.

shadycuz commented 6 years ago

We will finalize the PR soon and I will push a new package to pypy and release a new version.

On Wed, Jun 6, 2018, 7:14 PM Mark Hahn notifications@github.com wrote:

That was quick. Thanks to all. I assume I'll get a notification in octoprint when a new version is out.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/taxilian/OctoPrint-Twilio/issues/10#issuecomment-395270693, or mute the thread https://github.com/notifications/unsubscribe-auth/AQTJt0AAMQhMOs3uu3ecKle_hrKymr2fks5t6IyMgaJpZM4UdR_n .

shadycuz commented 6 years ago

@mark-hahn @tedder The new version with the proper sarge package was pushed to pypi. I'm not sure if on update it will handle the new package dependency correctly. You may have to remove and reinstall Octoprint-Twilio. Re-open this issue if you have further problems =).

mark-hahn commented 6 years ago

thks

On Sat, Jun 9, 2018 at 3:06 PM, Levi Blaney notifications@github.com wrote:

@mark-hahn https://github.com/mark-hahn @tedder https://github.com/tedder The new version with the proper sarge package was pushed to pypi. I'm not sure if on update it will handle the new package dependency correctly. You may have to remove and reinstall Octoprint-Twilio. Re-open this issue if you have further problems =).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/taxilian/OctoPrint-Twilio/issues/10#issuecomment-396001990, or mute the thread https://github.com/notifications/unsubscribe-auth/AAxhv0ot3SAFGK-ymQbBBcglmbYas2nvks5t7Eb2gaJpZM4UdR_n .