<div dir="ltr"><div>Well, this is interesting, I'm looking forward how this will develop.<br><br>Here is code review of some minor issues.<br></div><div><br>On Fri, Mar 13, 2015 at 1:24 PM, <<a href="mailto:svn_grass@osgeo.org">svn_grass@osgeo.org</a>> wrote:<br>><br>> +    def loadSettings(self, sett=None):<br>> +        if sett:<br>> +            self.settings = sett<br>> +<br>> +        try:<br>> +            self.database.SetValue(self.settings['database'])<br>> +        except:<br>> +            print 'err'<br>> +            pass<br>> +        try:<br>> +            self.schema.SetValue(self.settings['schema'])<br>> +        except:<br>> +            pass<br>> +        try:<br>> +            self.host.SetValue(self.settings['host'])<br>> +        except:<br>> +            pass<br>> +        try:<br>> +            self.user.SetValue(self.settings['user'])<br>> +        except:<br>> +            pass<br>> +        try:<br>> +            self.port.SetValue(self.settings['port'])<br>> +        except:<br>> +            pass<br>> +        try:<br>> +            self.passwd.SetValue(self.settings['passwd'])<br>> +        except:<br>> +            pass<br><br>This should be one big try-except. Also, `except:` is a bad practice, this will catch everything including perhaps even SyntaxError. You should specify the error/exception type. You should also specify in the comment why it is OK to ignore the error. What you should use in case of `<span class="">eval</span><span class=""></span>` that's a questions, more checks with `eval` is always a good idea. <br><br></div>Also, please use<br><br>if __name__ == '__main__':<br><br>for consistency with other modules. This will allow you to import the files and avoid code duplication between wx.*.py file and g.gui.*.py in case the wx.*.py file is needed.<br><br>Try to check your files with  pep8 before submitting.<br></div>