Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move imports to top for pjlink #29540

Merged
merged 1 commit into from Dec 6, 2019
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
8 changes: 3 additions & 5 deletions homeassistant/components/pjlink/media_player.py
@@ -1,9 +1,11 @@
"""Support for controlling projector via the PJLink protocol."""
import logging

from pypjlink import MUTE_AUDIO, Projector
from pypjlink.projector import ProjectorError
import voluptuous as vol

from homeassistant.components.media_player import MediaPlayerDevice, PLATFORM_SCHEMA
from homeassistant.components.media_player import PLATFORM_SCHEMA, MediaPlayerDevice
from homeassistant.components.media_player.const import (
SUPPORT_SELECT_SOURCE,
SUPPORT_TURN_OFF,
Expand Down Expand Up @@ -90,15 +92,13 @@ def __init__(self, host, port, name, encoding, password):

def projector(self):
"""Create PJLink Projector instance."""
from pypjlink import Projector

projector = Projector.from_address(self._host, self._port, self._encoding)
projector.authenticate(self._password)
return projector

def update(self):
"""Get the latest state from the device."""
from pypjlink.projector import ProjectorError

with self.projector() as projector:
try:
Expand Down Expand Up @@ -171,8 +171,6 @@ def turn_on(self):
def mute_volume(self, mute):
"""Mute (true) of unmute (false) media player."""
with self.projector() as projector:
from pypjlink import MUTE_AUDIO

projector.set_mute(MUTE_AUDIO, mute)

def select_source(self, source):
Expand Down