cleanup based on flake8 complaints

sjswerdloff · Jun 18, 2024 · c85fbe6 · c85fbe6
1 parent c9dd33e
commit c85fbe6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 11 deletions.
diff --git a/tdwii_plus_examples/TDWII_PPVS_subscriber/ppvs_subscriber_widget.py b/tdwii_plus_examples/TDWII_PPVS_subscriber/ppvs_subscriber_widget.py
@@ -8,13 +8,7 @@
 from ppvsscp import PPVS_SCP
 from pydicom.valuerep import VR
 from PySide6.QtCore import QDateTime, Qt, Slot  # pylint: disable=no-name-in-module
-from PySide6.QtWidgets import (
-    QApplication,
-    QFileDialog,
-    QTreeWidget,
-    QTreeWidgetItem,
-    QWidget,
-)
+from PySide6.QtWidgets import QApplication, QFileDialog, QTreeWidgetItem, QWidget
 
 # Important:
 # You need to run the following command to generate the ui_form.py file
@@ -112,7 +106,6 @@ def _restart_scp(self):
     @Slot()
     def _get_ups(self, ups_uid: str = ""):
         # do C-FIND-RQ
-        ups_responses = list()
         my_ae_title = self.ui.ppvs_ae_line_edit.text()
         upsscp_ae_title = self.ui.ups_ae_line_edit.text()
         machine_name = self.ui.machine_name_line_edit.text()

diff --git a/tdwii_plus_examples/TDWII_PPVS_subscriber/upsfindscu.py b/tdwii_plus_examples/TDWII_PPVS_subscriber/upsfindscu.py
@@ -1,12 +1,11 @@
 import logging
-from datetime import datetime
 from typing import Iterator
 
 import pydicom
 import pynetdicom
 from pydicom import DataElement, Dataset
 from pydicom.valuerep import VR
-from pynetdicom import AE, UnifiedProcedurePresentationContexts
+from pynetdicom import UnifiedProcedurePresentationContexts
 from pynetdicom.sop_class import UnifiedProcedureStepPull
 
 import tdwii_plus_examples.tdwii_config as ae_config
@@ -77,7 +76,8 @@ def create_ups_query(
 
     Args:
         ups_uid (str): The SOP Instance UID for the specific Unified Procedure Step. Empty default
-        machine_name (str): The name of the treatment machine as specified in the RT (Ion) Plan. Empty default but should be populated.
+        machine_name (str): The name of the treatment machine as specified in the RT (Ion) Plan.
+            Empty default but should be populated.
         procedure_step_state (str): SCHEDULED/IN PROGRESS/COMPLETED/CANCELED, "SCHEDULED" default
         scheduled_no_sooner_than (datetime):  earliest possible matching start datetime. Empty default
         scheduled_no_later_than (datetime): latest possible matching start datetime. Empty default