raising AttributeError in __getattr__ instead of NameError is the correct way of handling non-existent attributes; otherwise for example hasattr(obj, attribute) would raise NameError instead of returning False if the attribute does not exist

pyqtgraph/graphicsWindows.py

@@ -48,7 +48,7 @@ class TabWindow(QtGui.QMainWindow):
         if hasattr(self.cw, attr):
             return getattr(self.cw, attr)
         else:
-            raise NameError(attr)
+            raise AttributeError(attr)
     
 
 class PlotWindow(PlotWidget):

pyqtgraph/widgets/PlotWidget.py

@@ -76,7 +76,7 @@ class PlotWidget(GraphicsView):
             m = getattr(self.plotItem, attr)
             if hasattr(m, '__call__'):
                 return m
-        raise NameError(attr)
+        raise AttributeError(attr)
     
     def viewRangeChanged(self, view, range):
         #self.emit(QtCore.SIGNAL('viewChanged'), *args)