Configurator Error

I just wanted to report an error I ran into with the configurator.py. I'm trying to get an updated plasmac simulation running. This isn't actually connected to my machine, let me know if this is the root of the configurator's problem here.

I'm using linuxcnc 2.9.0-pre0-... from buildbot master.

All I've done for config so far is use the Sample Configurations -> by_machine -> plasmac -> gmoccapy -> imperial_plasmac

This runs fine for the simulator. However I don't have the shapes stuff in the extras tab. So I attempted to run the configurator on this imperial_plasmac.ini

Python throws the following error after hitting upgrade:
Also, just an FYI, I noticed that if the user closes the file picker dialog box without picking a file then python throws an error and then freezes. I just wanted to mention it in case you wanted to throw in some error handling there.

Thanks for the report.

I have never tried to upgrade a sample config because I only use Run In Place and it doesn't copy the sample config over. I will give it a try.

Error checking is the bane of my life...

All I've done for config so far is use the Sample Configurations -> by_machine -> plasmac -> gmoccapy -> imperial_plasmac

[strike]Would you be able to zip this complete config and post it here?[/strike]

Edit: I have pushed what I "think" is a fix for these issues.

phillc54 wrote:
Error checking is the bane of my life...

A professor of electronics in Germany told a friend of mine:
Electronics is 10% work, 90% troubleshooting that same work !
Luckily i had it in reverse! But then there is reverse engineering of electronics........... spent the day doing just that !

tommylight wrote: A professor of electronics in Germany told a friend of mine:
Electronics is 10% work, 90% troubleshooting that same work !

I think it might be a similar ratio for software, at least it seems it is for me.

Heads up Phill, looks like a recent update messed up some of the indention in the configurator.py around line #801.
forum.linuxcnc.org/9-installing-linuxcnc/38051-plasmac

Why does python rely on indentation levels anyway? Never understood that...

AgentWD40 wrote: Heads up Phill, looks like a recent update messed up some of the indention in the configurator.py around line #801.
forum.linuxcnc.org/9-installing-linuxcnc/38051-plasmac

Thanks Kyle, I have just pushed the fixes.

Why does python rely on indentation levels anyway? Never understood that...

I think it is supposed to be easier to spot errors if it is nice and tidy, but that obviously doesn't help me.

AgentWD40 wrote: Why does python rely on indentation levels anyway? Never understood that...

I think its supposed to save some keystrokes for developers becasue it gets away from {} and if..end
But generally if you open a file in a different editor it breaks the indentation...

I think I may have discovered a new error with configurator.py. I found it when trying to reconfigure an existing setup. See attached picture.

To replicate: Run configurator, click "Reconfigure", change radio button for THC Modes. Error results.

Maybe I did something wrong?

Should be OK in the latest version.