Can't save printer with Configtool

[shakenov-chinga]

Hi.
I'm trying to save loaded and edited printer configurations. This error message is printed in terminal whenever I save, no matter if it is editted or not.

Traceback (most recent call last):
File "/home/chinga/Dropbox/Teacup_Firmware-master/configtool/gui.py", line 246, in onSavePrinterConfig
rc = self.pgPrinter.onSaveConfig(evt)
File "/home/chinga/Dropbox/Teacup_Firmware-master/configtool/printerpanel.py", line 198, in onSaveConfig
return self.saveConfigFile(path)
File "/home/chinga/Dropbox/Teacup_Firmware-master/configtool/printerpanel.py", line 246, in saveConfigFile
v1 = pg.getValues()
File "/home/chinga/Dropbox/Teacup_Firmware-master/configtool/mechanicalpage.py", line 242, in getValues
result['HOMING_STEP4'] = self.getHomingValue('HOMING_STEP4', result)
File "/home/chinga/Dropbox/Teacup_Firmware-master/configtool/mechanicalpage.py", line 231, in getHomingValue
return (self.labels.keys()[self.labels.values().index(result[name][0])], True) ValueError: 'n' is not in list

[Wurstnase]

I can confirm this issue. Looks like the button with '-' will return something different than expected. Also for other homing-keys if they are '-'. E.g. for Step 4 it is 'n'. For Step 2 it is 'y'.

[Wurstnase]

Ok, I found what happened.

I load printer.mendel.h. Want to save it as e.g. printer.test.h. Now the DEFINE_HOMING looks like:

/** \def DEFINE_HOMING

  Order (and number) of homing movements.
*/
#ifndef DEFINE_HOMING
  #define DEFINE_HOMING(...)
#endif
//DEFINE_HOMING_START
DEFINE_HOMING(('x', False), ('y_negative', True), ('z_negative', True), ('n', False))
//DEFINE_HOMING_END

[Wurstnase]

Uff... Lot's of work.
Please check https://github.com/Traumflug/Teacup_Firmware/tree/configtool_homing
Feedback is welcome.

Homing is fixed. Also this should work with Python3 now.

I'm currently not very happy, with the double defines of homingKeys in mechanicalpage.py and printer.py. But I don't have any good idea for this.

[sv1]

With python 2.7.13 on Debian, I can not run configtool at all.

$ ./configtool.py
Traceback (most recent call last):
File "./configtool.py", line 191, in
StartGui(getSettings())
File "./configtool.py", line 38, in getSettings
settings = Settings(None, cmdFolder, arg)
File "/opt/linux/Teacup_Firmware/configtool/settings.py", line 53, in init
self.loaded = self.readConfig(ini)
File "/opt/linux/Teacup_Firmware/configtool/settings.py", line 78, in readConfig
self.cflags = cfg_get("cflags")
File "/opt/linux/Teacup_Firmware/configtool/settings.py", line 73, in cfg_get
val = cfg.get(key, values[key])
File "/usr/lib/python2.7/dist-packages/configparser.py", line 1302, in get
fallback=fallback)
File "/usr/lib/python2.7/dist-packages/configparser.py", line 816, in get
d)
File "/usr/lib/python2.7/dist-packages/configparser.py", line 401, in before_get
self._interpolate_some(parser, option, L, value, section, defaults, 1)
File "/usr/lib/python2.7/dist-packages/configparser.py", line 450, in _interpolate_some
u"found: %r" % (rest,))
configparser.InterpolationSyntaxError: '%' must be followed by '%' or '(', found: u'%F_CPU% -mmcu=%CPU% -Wall -Wstrict-prototypes -std=gnu99 -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -Winline -fno-move-loop-invariants -fno-tree-scev-cprop -Os -ffunction-sections -finline-functions-called-once -mcall-prologues -Wa,-adhlns=%ALNAME% -save-temps=obj'

[Wurstnase]

I've currently not much time to do any support but probably this issue was already fixed in
https://github.com/Traumflug/Teacup_Firmware/tree/python_upgrade .

Can you please check?

[sv1]

On Sat, 2019-04-06 at 11:12 -0700, Nico Tonnhofer wrote: I've currently not much time to do any support but probably this issue was already fixed in https://github.com/Traumflug/Teacup_Firmware/tree/python_upgrade . Can you please check?

Same behavior .... $ git checkout python_upgrade Branch python_upgrade set up to track remote branch python_upgrade from origin. Switched to a new branch 'python_upgrade' $ ./configtool.py  Traceback (most recent call last):   File "./configtool.py", line 191, in <module>     StartGui(getSettings())   File "./configtool.py", line 38, in getSettings     settings = Settings(None, cmdFolder, arg)   File "/opt/linux/Teacup_Firmware/configtool/settings.py", line 53, in __init__     self.loaded = self.readConfig(ini)   File "/opt/linux/Teacup_Firmware/configtool/settings.py", line 78, in readConfig     self.cflags = cfg_get("cflags")   File "/opt/linux/Teacup_Firmware/configtool/settings.py", line 73, in cfg_get     val = cfg.get(key, values[key])   File "/usr/lib/python2.7/dist-packages/configparser.py", line 1302, in get     fallback=fallback)   File "/usr/lib/python2.7/dist-packages/configparser.py", line 816, in get     d)   File "/usr/lib/python2.7/dist-packages/configparser.py", line 401, in before_get     self._interpolate_some(parser, option, L, value, section, defaults, 1)   File "/usr/lib/python2.7/dist-packages/configparser.py", line 450, in _interpolate_some     u"found: %r" % (rest,)) configparser.InterpolationSyntaxError: '%' must be followed by '%' or '(', found: u'%F_CPU% -mmcu=%CPU% -Wall -Wstrict-prototypes -std=gnu99 -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -Winline -fno-move-loop-invariants -fno-tree-scev-cprop -Os -ffunction- sections -finline-functions-called-once -mcall-prologues -Wa,- adhlns=%ALNAME% -save-t

[sv1]

On Sat, 2019-04-06 at 11:12 -0700, Nico Tonnhofer wrote: I've currently not much time to do any support but probably this issue was already fixed in https://github.com/Traumflug/Teacup_Firmware/tree/python_upgrade . Can you please check? —

Started with a new/clean repo and now I have this: $ git checkout configtool_homing Branch configtool_homing set up to track remote branch configtool_homing from origin. Switched to a new branch 'configtool_homing' $ ./configtool.py  Traceback (most recent call last):   File "./configtool.py", line 191, in <module>     StartGui(getSettings())   File "/opt/linux/Teacup_Firmware/configtool/gui.py", line 695, in StartGui     frame = ConfigFrame(settings)   File "/opt/linux/Teacup_Firmware/configtool/gui.py", line 77, in __init__     self.pgPrinter = PrinterPanel(self, self.nb, self.settings)   File "/opt/linux/Teacup_Firmware/configtool/printerpanel.py", line 54, in __init__     self.pgMech = self.registerPage(MechanicalPage, "Mechanical")   File "/opt/linux/Teacup_Firmware/configtool/printerpanel.py", line 65, in registerPage     self, self.nb, len(self.pages), *args, font=self.settings.font, **kwargs   File "/opt/linux/Teacup_Firmware/configtool/mechanicalpage.py", line 175, in __init__     b.SetToolTip("Open the calculator for axes that are belt-driven.")   File "/usr/lib/python2.7/dist-packages/wx-3.0-gtk2/wx/_core.py", line 11390, in SetToolTip     return _core_.Window_SetToolTip(*args, **kwargs) TypeError: in method 'Window_SetToolTip', expected argument 2 of type 'wxToolTip *'

[Wurstnase]

Please install the latest wxpython: https://wxpython.org/pages/downloads/index.html
wx3.0 is not supported with python_upgrade-branch.

[Wurstnase]

Everything now in the experimental branch.