Skip to content

Commit

Permalink
Small fixes
Browse files Browse the repository at this point in the history
- address style warnings
- tune console output at the end of the process
  • Loading branch information
marcelstoer committed Oct 2, 2017
1 parent db84ce5 commit a6aae0c
Showing 1 changed file with 13 additions and 10 deletions.
23 changes: 13 additions & 10 deletions Main.py
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,8 @@ def run(self):
try:
esp.change_baud(self._config.baud)
except NotImplementedInROMError:
print("WARNING: ROM doesn't support changing baud rate. Keeping initial baud rate %d" % initial_baud)
print("WARNING: ROM doesn't support changing baud rate. Keeping initial baud rate %d." %
initial_baud)

args = Namespace()
args.flash_size = "detect"
Expand All @@ -96,6 +97,8 @@ def run(self):
if self._config.erase_before_flash:
esptool.erase_flash(esp, args)
esptool.write_flash(esp, args)
# The last line printed by esptool is "Leaving..." -> some indication that the process is done is needed
print("\nDone.")
except SerialException as e:
self._parent.report_error(e.strerror)
raise e
Expand Down Expand Up @@ -219,12 +222,12 @@ def on_pick_file(event):

baud_boxsizer = wx.BoxSizer(wx.HORIZONTAL)

def add_baud_radio_button(sizer, idx, rate):
style = wx.RB_GROUP if idx == 0 else 0
radio_button = wx.RadioButton(panel, name="baud-%d" % rate, label="%d" % rate, style=style)
radio_button.rate = rate
def add_baud_radio_button(sizer, index, baud_rate):
style = wx.RB_GROUP if index == 0 else 0
radio_button = wx.RadioButton(panel, name="baud-%d" % baud_rate, label="%d" % baud_rate, style=style)
radio_button.rate = baud_rate
# sets default value
radio_button.SetValue(rate == self._config.baud)
radio_button.SetValue(baud_rate == self._config.baud)
radio_button.Bind(wx.EVT_RADIOBUTTON, on_baud_changed)
sizer.Add(radio_button)
sizer.AddSpacer(10)
Expand All @@ -234,8 +237,8 @@ def add_baud_radio_button(sizer, idx, rate):

flashmode_boxsizer = wx.BoxSizer(wx.HORIZONTAL)

def add_flash_mode_radio_button(sizer, idx, mode, label):
style = wx.RB_GROUP if idx == 0 else 0
def add_flash_mode_radio_button(sizer, index, mode, label):
style = wx.RB_GROUP if index == 0 else 0
radio_button = wx.RadioButton(panel, name="mode-%s" % mode, label="%s" % label, style=style)
radio_button.Bind(wx.EVT_RADIOBUTTON, on_mode_changed)
radio_button.mode = mode
Expand All @@ -249,8 +252,8 @@ def add_flash_mode_radio_button(sizer, idx, mode, label):

erase_boxsizer = wx.BoxSizer(wx.HORIZONTAL)

def add_erase_radio_button(sizer, idx, erase_before_flash, label, value):
style = wx.RB_GROUP if idx == 0 else 0
def add_erase_radio_button(sizer, index, erase_before_flash, label, value):
style = wx.RB_GROUP if index == 0 else 0
radio_button = wx.RadioButton(panel, name="erase-%s" % erase_before_flash, label="%s" % label, style=style)
radio_button.Bind(wx.EVT_RADIOBUTTON, on_erase_changed)
radio_button.erase = erase_before_flash
Expand Down

0 comments on commit a6aae0c

Please sign in to comment.