From 94c1ebc8e943983a7e04d2fc03ba4b69a03d904f Mon Sep 17 00:00:00 2001
From: stevenhua0320 <r.hua@mail.utoronto.ca>
Date: Thu, 25 Sep 2025 16:18:13 -0400
Subject: [PATCH] Fix module not used, module not on top level& py2 cond
 deprecation

---
 src/diffpy/srxplanargui/srxconfig.py | 78 +++-------------------------
 src/diffpy/srxplanargui/srxgui.py    | 63 ++++------------------
 2 files changed, 16 insertions(+), 125 deletions(-)

diff --git a/src/diffpy/srxplanargui/srxconfig.py b/src/diffpy/srxplanargui/srxconfig.py
index 0dd3d57..b734e44 100644
--- a/src/diffpy/srxplanargui/srxconfig.py
+++ b/src/diffpy/srxplanargui/srxconfig.py
@@ -13,88 +13,24 @@
 #
 ##############################################################################
 
-import argparse
-import configparser
+
 import os
-import re
-import sys
-from functools import partial
 
 import numpy as np
-from traits.etsconfig.api import ETSConfig
-
-ETSConfig.toolkit = "qt4"
-
 from diffpy.srxconfutils.configtraits import ConfigBaseTraits
-from diffpy.srxconfutils.tools import (
-    _configPropertyR,
-    _configPropertyRad,
-    _configPropertyRW,
-)
 from diffpy.srxplanar.srxplanarconfig import (
-    _defaultdata,
     _description,
     _epilog,
     _optdatalist,
     checkMax,
 )
 from pyface.api import ImageResource
-from traits.api import (
-    Any,
-    Array,
-    Bool,
-    Button,
-    CFloat,
-    CInt,
-    DelegatesTo,
-    Dict,
-    Directory,
-    Enum,
-    Event,
-    File,
-    Float,
-    HasTraits,
-    Instance,
-    Int,
-    List,
-    Property,
-    Range,
-    Str,
-    cached_property,
-    on_trait_change,
-    property_depends_on,
-)
-from traitsui.api import (
-    Action,
-    ArrayEditor,
-    BooleanEditor,
-    ButtonEditor,
-    CheckListEditor,
-    Controller,
-    EnumEditor,
-    Group,
-    Handler,
-    HGroup,
-    HistoryEditor,
-    InstanceEditor,
-    Item,
-    RangeEditor,
-    Tabbed,
-    TableEditor,
-    TextEditor,
-    TitleEditor,
-    VGroup,
-    View,
-    spring,
-)
-from traitsui.menu import (
-    CancelButton,
-    Menu,
-    MenuBar,
-    OKButton,
-    OKCancelButtons,
-    ToolBar,
-)
+from traits.api import Bool, Enum, Property, on_trait_change
+from traits.etsconfig.api import ETSConfig
+from traitsui.api import Group, Item, View
+
+ETSConfig.toolkit = "qt"
+
 
 _optdatalist.append(
     [
diff --git a/src/diffpy/srxplanargui/srxgui.py b/src/diffpy/srxplanargui/srxgui.py
index 1f54e0c..d3e7d21 100644
--- a/src/diffpy/srxplanargui/srxgui.py
+++ b/src/diffpy/srxplanargui/srxgui.py
@@ -16,75 +16,30 @@
 import os
 import sys
 
-import numpy as np
-from traits.etsconfig.api import ETSConfig
-
-ETSConfig.toolkit = "qt4"
-
 from diffpy.srxplanar.srxplanar import SrXplanar
-from pyface.api import ImageResource, SplashScreen
-from traits.api import (
-    Any,
-    Array,
-    Bool,
-    Button,
-    CFloat,
-    CInt,
-    DelegatesTo,
-    Dict,
-    Directory,
-    Enum,
-    Event,
-    File,
-    Float,
-    HasTraits,
-    Instance,
-    Int,
-    List,
-    Property,
-    Range,
-    Str,
-    cached_property,
-    on_trait_change,
-    property_depends_on,
-)
+from pyface.api import ImageResource
+from traits.api import Any, Button, File, HasTraits, Instance
+from traits.etsconfig.api import ETSConfig
 from traitsui.api import (
     Action,
-    ArrayEditor,
-    ButtonEditor,
-    CheckListEditor,
-    Controller,
-    EnumEditor,
     Group,
     Handler,
     HGroup,
-    HistoryEditor,
-    ImageEditor,
     InstanceEditor,
     Item,
-    RangeEditor,
-    Tabbed,
-    TableEditor,
-    TextEditor,
-    TitleEditor,
     VGroup,
     View,
     spring,
 )
-from traitsui.menu import (
-    CancelButton,
-    Menu,
-    MenuBar,
-    OKButton,
-    OKCancelButtons,
-    ToolBar,
-)
+from traitsui.menu import CancelButton, OKButton
 
 from diffpy.srxplanargui.calibration import Calibration
 from diffpy.srxplanargui.help import SrXguiHelp
 from diffpy.srxplanargui.selectfiles import AddFiles
 from diffpy.srxplanargui.srxconfig import SrXconfig
 
+ETSConfig.toolkit = "qt"
+
 
 class SrXguiHandler(Handler):
 
@@ -180,7 +135,7 @@ def processSelected(self, summation=False):
     def detectConfigfile(self, filename):
         """Current directory > home directory, if none, then return the
         curdir+filename if 'default', then return home+filename."""
-        if filename == None:
+        if filename is None:
             configfile = os.path.join(os.path.curdir, "srxconfig.cfg")
         elif filename == "default":
             configfile = os.path.join(os.path.expanduser("~"), "srxconfig.cfg")
@@ -243,11 +198,11 @@ def _helpbb_changed(self):
 
     def _selfcalibratebb_changed(self):
         image = None
-        if self.addfiles.selected != None:
+        if self.addfiles.selected is not None:
             if len(self.addfiles.selected) == 1:
                 image = self.addfiles.selected[0].fullname
 
-        if image != None:
+        if image is not None:
             self.calibration.image = image
         self.calibration.edit_traits(view="main_View")
         return